Proof of concept for IP checking

Lots if refactoring to do, but concept is working.
If you have any confirmed IP addresses used by crawlers, please feel free to add them in this PR.

It may be useful to use a library to check the IPs iin ranges such as https://github.com/wikimedia/ipset for speed. I've done some testing with this, and it's a lot faster than a simple, brute-force lookup.
95%+ of the usefulness of any such functionality would always be with a good and regularly updated source of IPs, crawlers, and other potential sources (like something that has caused me some problems recently - spam mailing list requests from TOR exit nodes), and being able to select one or more sources at will to check against.

@alister Thanks for the suggested IP checker. I will definitely benchmark that later down the line. Also, I have seen Symfony have IP utilities in the http-foundation package, so will test that too.
We are very much taking baby steps with this, as we are aware of the potential maintainability issues with IP lists. I feel they would need to be checked way more than user agents need to be.

Gonna hold off with it for the time being.
I may reassess in the future.
If anyone fancies progressing this in the meantime, then all PR will be welcomed.
Thanks


tldts-utils uses deprecated node punycode module

replace import in parser.ts with
import { toASCII } from 'punycode/';

and add punycode to dependencies!

Hey,
Thanks for reaching out. Here is the fix: https://github.com/remusao/tldts/pull/1887


fix(login): update automatic registration to pull form data

Resolves https://github.com/zitadel/zitadel/issues/4090#issue-1324935992
I confirmed that this allows users to register and resolves at this the issue. Not sure if there are other tests or flows that need to be tested. Open to feedback on whether this is the right place to put this change.

This also relates to #4089

KUDOS to @cyb3rd0g1 for this PR.
Let us review this and come back with feedback.

Updated to resolve #4089 as well since the change is pretty simple
https://github.com/zitadel/zitadel/pull/4103/commits/bfd11829d1300ff1b2e4dde196a90582a92d2fc5

Makes sense to me, thanks for the ffeedback.


Add a yielding filter

It seems like a yielding filter would be useful to add to Accumulo.  Could add an option like yield.time to the existing filter base class.  The downside of adding an option to the existing class is that it may conflict with user filter options.   Could also create a new class, but that option would not benefit existing implementations of Filter.
Below is a link to a yielding filter I created for performance testing for apache/accumulo-testing#56
YieldingFilter.java

There is also the YieldingIerator


Shell completion fails in CentOS

[X ] I have read through the manual page (man fzf)
[X ] I have the latest version of fzf
[ X] I have searched through the existing issues

Info

OS

[ X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
source ${FZF}/shell/completion.bash
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  nospace -o dirnames: invalid option name
bash: complete:  default -o nospace -v: invalid option name
bash: complete:  default -o nospace -v: invalid option name
bash: complete:  default -o bashdefault: invalid option name
bash: complete:  default -o bashdefault: invalid option name
Possible reason: if constructs like this
if ! declare -f _fzf_compgen_dir > /dev/null; then

Hi @sivaswami,
Could you add some more information to your problem? I'm finding it hard to understand what you mean by that, maybe by now you've fixed it?

It's probably because your bash version is too low. Make sure to use bash 3 or above (preferably 4 or above).

I had the same issue on an up-to-date Arch Linux, bash v5.1.16
It's an IFS issue I believe, the whole shebang after -o is interpreted as one string. Solved it with eval:
diff --git a/shell/completion.bash b/shell/completion.bash
index 9474ea0..6096a05 100644
--- a/shell/completion.bash
+++ b/shell/completion.bash
@@ -337,7 +337,7 @@ __fzf_defc() {
printf -v def "$orig" "$func"
eval "$def"
else

complete -F "$func" $opts "$cmd"

eval "complete -F "$func" $opts "$cmd""
fi
}


CCO Governance Dashboard

I am unable to load/use the new v2 CCO Governance dashboard.   It is failing on :
= Table.RemoveColumns(blueprints,{"id", "tenantId", "countryCode", "displayName", "domains", "tenantCategory", "ManagementGroups.count"})
Expression.Error: The column 'id' of the table wasn't found.
Details:
id
It looks like the data being returned from Source into the various tables is not in line with what is expected.  Anyone else having this issue?     v8 of the inf dashboard is working well thx.

Hi @CeeeBeee ,
Do you have Management Groups in your environment or the correct level of permissions to retrieve this data?

Yes,
We use management groups, I created and published one blueprint (as it is the blueprints transforms that are failing).  And yes I am a global admin across Azure.
Anyother thoughts?
Thanks
Colin
From: Jordi Suñé<EMAIL_ADDRESS>Sent: 15 July 2020 08:26
To: Azure/ccodashboard<EMAIL_ADDRESS>Cc: Colin Bradford<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/ccodashboard] CCO Governance Dashboard (#104)
Hi @CeeeBeee , Do you have Management Groups in your environment or the correct
Email scanned - Sender unknown | CORVID Email Protectionhttps://corvid.it/cyber-solutions/email-protection/
⚠ Replying may not go to displayed sender
View email infohttps://pernixconsole.corvid.it/emaildetailbrief/67A9EE90-C66C-11EA-A3AF-C9F6951E80B0  |  View all emailshttps://pernixconsole.corvid.it/genotl  |  Report as suspiciousmailto:pernixsus@ultra-cpg.com?subject=Reporting Suspicious Pernix Email&body=The email 67A9EE90-C66C-11EA-A3AF-C9F6951E80B0 is suspicious  |   Sender<EMAIL_ADDRESS>Hi @CeeeBeeehttps://github.com/CeeeBeee ,
Do you have Management Groups in your environment or the correct level of permissions to retrieve this data?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/ccodashboard/issues/104#issuecomment-658595639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APCRRVK7R2B44LRUIUL6E4TR3VKXXANCNFSM4OYPLQ6Q.
This e-mail from Blue Prism Group plc or a member of its group of companies is intended exclusively for the addressee and may contain legally privileged or confidential information. If you are not the intended recipient you must not read use nor disclose the information in the email or attachments. Please notify Blue Prism by replying to this email and permanently delete the email from your records. Blue Prism operates an anti-virus policy but does not accept responsibility for any damage caused by viruses transmitted. Blue Prism Group plc Registered in England No: 09759493. Registered Office: 2 Cinnamon Park, Warrington, Cheshire, WA2 0XP

Let's troubleshoot this issue offline. Will ping you by email

New version of the dashboard fixing this issue was released (v2.1), please check https://github.com/Azure/ccodashboard/tree/master/dashboards/CCODashboard-Governance

Jordi
That update did not work for me.     Can we speak tomorrow / Tuesday to review and understand where the issue is.    Also, for the infrastructure dashboard, I am no seeing recent security incidents – maybe I can show you at the same time.
Best regards
Colin Bradford
Chief Operating Officer
<EMAIL_ADDRESS>D: +44 (0)7545374611
cloud.blueprism.comhttps://cloud.blueprism.com/
From: Jordi Suñé<EMAIL_ADDRESS>Sent: 30 July 2020 09:24
To: Azure/ccodashboard<EMAIL_ADDRESS>Cc: Colin Bradford<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/ccodashboard] CCO Governance Dashboard (#104)
New version of the dashboard fixing this issue was released (v2.1), please chec
Email scanned - Sender unknown | CORVID Email Protectionhttps://corvid.it/cyber-solutions/email-protection/
⚠ Replying may not go to displayed sender
View email infohttps://pernixconsole.corvid.it/emaildetailbrief/FC9CC9E6-D23D-11EA-AF8C-543372C153D9  |  View all emailshttps://pernixconsole.corvid.it/genotl  |  Report as suspiciousmailto:pernixsus@ultra-cpg.com?subject=Reporting Suspicious Pernix Email&body=The email FC9CC9E6-D23D-11EA-AF8C-543372C153D9 is suspicious  |   Sender<EMAIL_ADDRESS>New version of the dashboard fixing this issue was released (v2.1), please check https://github.com/Azure/ccodashboard/tree/master/dashboards/CCODashboard-Governance
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/ccodashboard/issues/104#issuecomment-666221740, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APCRRVLUK6ITKF7CYAHFUULR6EUZBANCNFSM4OYPLQ6Q.
This e-mail from Blue Prism Group plc or a member of its group of companies is intended exclusively for the addressee and may contain legally privileged or confidential information. If you are not the intended recipient you must not read use nor disclose the information in the email or attachments. Please notify Blue Prism by replying to this email and permanently delete the email from your records. Blue Prism operates an anti-virus policy but does not accept responsibility for any damage caused by viruses transmitted. Blue Prism Group plc Registered in England No: 09759493. Registered Office: 2 Cinnamon Park, Warrington, Cheshire, WA2 0XP


Add Latvian translation for messages fix level translation

Saw the settings update and decided to add Latvian translation right away. While playing through to recheck my previous translation, found that some more errors and fixed them as well.

@Akirtovskis, thanks for the speedy translations to the new settings!

Enjoy! Though the settings translation works perfectly, there is still some problem with level number 10 in Latvian version. I thought that adding the missing paragraph tag would fix it, but apparently it doesn't. From what I can see it looks equally formatted to other translations that work just as expected.

Looks like an unclosed code tag along with the p tag. I'll push a fix for it now.


ipa size is way huger then it should be

The ipa size is way huger then it should be. This project is really small. It does have too many images. But the ipa size is round 150M.

React Native version:
System:
OS: macOS 10.15.5
CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 178.86 MB / 8.00 GB
Shell: 5.8 - /usr/local/bin/zsh
Binaries:
Node: 10.15.3 - /usr/local/bin/node
Yarn: 1.21.1 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.6, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK:
API Levels: 28, 29
Build Tools: 28.0.3, 29.0.3
System Images: android-28 | Intel x86 Atom_64
Android NDK: Not Found
IDEs:
Android Studio: 4.0 AI-193.69<IP_ADDRESS>14223
Xcode: 11.6/11E708 - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_251 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found
Package.json
{
  "name": "name",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "bundle-ios": "react-native bundle --entry-file index.js  --platform ios --dev false --bundle-output ./ios/bundle/index.ios.jsbundle --assets-dest ./ios/bundle",
    "android": "react-native run-android",
    "ios": "react-native run-ios",
    "start": "react-native start",
    "test": "jest",
    "lint": "eslint ."
  },
  "dependencies": {
    "@invertase/react-native-apple-authentication": "^1.1.0",
    "@react-native-community/async-storage": "^1.11.0",
    "@react-native-community/masked-view": "^0.1.10",
    "@react-native-community/progress-view": "^1.0.3",
    "@react-navigation/bottom-tabs": "^5.5.2",
    "@react-navigation/native": "^5.5.1",
    "@react-navigation/stack": "^5.4.2",
    "axios": "^0.19.2",
    "moment": "^2.27.0",
    "prop-types": "^15.7.2",
    "react": "16.11.0",
    "react-native": "0.62.2",
    "react-native-actionsheet": "^2.4.2",
    "react-native-charts-wrapper": "^0.5.7",
    "react-native-dropdownalert": "^4.2.1",
    "react-native-gesture-handler": "^1.6.1",
    "react-native-htmlview": "^0.15.0",
    "react-native-keyboard-aware-scroll-view": "^0.9.1",
    "react-native-orientation": "^3.1.3",
    "react-native-pdf": "^6.1.2",
    "react-native-reanimated": "^1.9.0",
    "react-native-safe-area-context": "^3.0.2",
    "react-native-screens": "^2.8.0",
    "react-native-star-rating": "^1.1.0",
    "react-native-status-bar-height": "^2.5.0",
    "react-native-swiper": "^1.6.0",
    "react-native-tab-view": "^2.14.4",
    "react-native-webview": "^10.2.3",
    "react-native-wechat": "^1.9.12",
    "react-redux": "^7.2.0",
    "redux": "^4.0.5",
    "redux-axios-middleware": "^4.0.1",
    "redux-logger": "^3.0.6",
    "rn-fetch-blob": "^0.12.0"
  },
  "devDependencies": {
    "@babel/core": "^7.6.2",
    "@babel/runtime": "^7.10.2",
    "@react-native-community/eslint-config": "^0.0.5",
    "babel-jest": "^24.9.0",
    "eslint": "^6.5.1",
    "jest": "^24.9.0",
    "metro-react-native-babel-preset": "^0.58.0",
    "react-test-renderer": "16.11.0"
  },
  "jest": {
    "preset": "react-native"
  }
}

I uploaded it to TestFlight. The final size is 16M! Everything is fine.

I uploaded it to TestFlight. The final size is 16M! Everything is fine.

请问一下怎么解决的

@nino9012 不用解决，打包是100+M，其实APP就10+M。

Following solution for hermes enabled and RN 0.64.2.
1.Need to do bitcode enabled - Yes at Build setting tab in xcode.
2.Need to choose deployment target version above ios 13.1
Then once do clean & build and then archive. your .ipa file size will be reduce.


fix: spelling mistake correction

Correct spelling mistake in commonEditorConfig.ts file

Change word from mulitiplier to multiplier

Note:

Potentially affects "Get Started - User and Workspace Settings"
https://code.visualstudio.com/docs/getstarted/settings
page's listing of the default editor config parameters,
as those still include the (probably) generated mispelling.

Thanks @d2s !


Error in loading from Huggingface

When I try to run the following code in colab:
from datasets import load_dataset
dataset = load_dataset("xinrongzhang2022/InfiniteBench")
I get the following error:

DatasetGenerationCastError: An error occurred while generating the dataset
All the data files must have the same columns, but at some point there are 1 missing columns ({'options'})
This happened while the json dataset builder was generating data using
hf://datasets/xinrongzhang2022/InfiniteBench/kv_retrieval.jsonl (at revision 2c3c9fe62808833ab783026bbf8e7a47539a28c6)
Please either edit the data files to have matching columns, or separate them into different configurations (see docs at https://hf.co/docs/hub/datasets-manual-configuration#multiple-configurations)

We have re-uploaded the files and solved this problem. Please kindly use load_dataset("xinrongzhang2022/InfiniteBench") now!

Running into this issue as well

please kindly set features when loading datasets
import datasets
from dataset import Value, Sequence
ft = Features({"id": Value("int64"), "context": Value("string"), "input": Value("string"), "answer": Sequence(Value("string")), "options": Sequence(Value("string"))})
dataset = load_dataset("xinrongzhang2022/InfiniteBench", features=ft)

from datasets import load_dataset, Features, Value, Sequence
Define the features schema
ft = Features({
"id": Value("int64"),
"context": Value("string"),
"input": Value("string"),
"answer": Sequence(Value("string")),
"options": Sequence(Value("string"))
})
Load the dataset with the specified features
dataset = load_dataset("xinrongzhang2022/InfiniteBench", features=ft)

from datasets import load_dataset, Features, Value, Sequence
ft = Features({
"id": Value("int64"),
"context": Value("string"),
"input": Value("string"),
"answer": Sequence(Value("string")),
"options": Sequence(Value("string"))
})
dataset = load_dataset("xinrongzhang2022/InfiniteBench", features=ft)


"Invalid watch source" error if use vue-tweet in v-for

Using vue-tweet with v-for causing a "Invalid watch source".

This should be fixed here https://github.com/DannyFeliz/vue-tweet/releases/tag/v2.0.2

@DannyFeliz Issue still present. May changing props to ref would fix this. Like
    watch(toRef(props), () => {       renderTweet();     });

@mahapo I fixed in this PR https://github.com/DannyFeliz/vue-tweet/pull/28, sorry for being almost 2 years late 😅


Use Web Audio API to create guitar metronome sound effect

At the moment, it uses .wav audio file downloaded from internet for the guitar metronome sounds, not ideal. Best to use Web Audio API to do same.

Closing as this issue is for another repo.


#issue-02#error: The specified item could not be found in the keychain.

error: The specified item could not be found in the keychain.

原因:本地苹果证书丢失
文件: ~/Library/Keychains/login.keychain
~/Library/Keychains/metadata.keychain
~/Library/Keychains/metadata.keychain-db


Add azure-sdk to eng/common sync

After https://github.com/Azure/azure-sdk/pull/7201 azure-sdk will be dependent on changes from eng/common. It only depends on a few items but I decided it is better to add to eng/common instead of manually syncing or doing a reference.

/check-enforcer override


CORS

We are using https://api.ipify.org/?format=json. Which is working fine in MAC Chrome but in windows chrome it is GIVING CORS issue.

Disable adblock/ublock origin

If you enabled uBlock Origin, it will block the ipify.org domain. Disabling it on the site should resolve the CORS issue.

Not every user knows this and goes through steps. Any programmatic way to avoid this error?

If this is due to uBlock or similar plugins, what you have to do is to report it to those plugins as false positive site.


Move compose box buttons into text area

We should implement the part of the compose box redesign (#24878) that moves the compose box buttons into the text area, and changes how the scrolling works.
This should apply both to the compose box and the "edit message" UI.
This issue is part of #24878.

Live demo with the source code on Repl.it. Here is the decomposed source code archive:
compose-decomposed.zip
The css code is split to decomposed areas but all the js code is in a single file. The file isn't big so it is easier to follow the logic of interactions between html and js.
Not all interactions are handled. Prototype is focused on styles and new interactions which are not currently present.
For example it doesn't suggest TAB order but does suggest focus styles on different elements.

@zulipbot Claim

HI @alya, I will like to work on this.

As noted in the contributor guide, please choose an issue with the "help wanted" label.


Switch from ActionBar Spinner to Navigation Drawer

As discussed in #661. The current spinner navigation isn't really intuitive, and switching to a navigation drawer would be a good step forward. ActionBar's list navigation is deprecated now, so a switch would need to happen sooner or later anyway. This would also make the navigation more readily able to accept new areas to navigate to (settings, profile, etc.)
There's a couple good mockups in #368, I'll try to mock up a more material version tonight.

When i was playing around with my own version i made this

It's made with the material design guidelines in mind(some things need tweaking)

Ooooo very nice. I feel like we could make the header a little more material by displaying some information about the account, and with a different color/background. Also the list header color generally uses the accent color in MD iirc.
I was working on one as well but didn't really get a chance to work more on it until tonight. How does this look? (Ignore the wrong icons/font, I'm not that familiar with mockup tools :stuck_out_tongue_winking_eye:)

Looks good, merging the two would look great! Last night I say down an made some ground work for the navigation drawer, mostly turning old activities intro fragments. So if I have the time i might PR tonight

Cool cool

Looks awesome!
Finally, this app is on the verge of a UI upgrade it so deserves. =)
One thing, I was thinking about making the header in the nav drawer to be more compact.
Perhaps like this:

What do you guys think?

Yeah that would make sense since there's not really a user "cover photo" that we could show there or anything

+1 for compact header

It looks like i might have to update the Octicons because the home icon was added later. This means that some of the icons will be changed since they are no longer there. But I will also mean that we will have the same icons as the website. Any objections?

@Meisolsson are you working on a mockup or actively developing this?

Developing for the moment

ok good to know, please link your fork/branch.
It would be nice if GitHub could provide these icons. @fadils?

Octicons is the GitHub icons made by GitHub

I know, I just mean it would be convenient if they gave us official icons at the right resolutions so we don't have to go hunting ourselves :smiley:

Well yeah, but since it's a font it cab scale to the perfect size and it has the same icon as the website. So adding drawables would be worse

Are we talking about font or icons?

It's an icon font (https://github.com/github/octicons)

Ah I see. Well, I'd wait for one of them to chime in then, I'm not familiar with this.

No problems, just wanted to ask. I'll push to my fork later so you can see the progress

No problems with the icons so please proceed..

@hzsweers The black background that you use in you mock would look great. Could you send a copy?

So this is almost ready for a PR: http://imgur.com/uDSHpOv
Any ideas on what de should put under the username if the user doesn't have their email public?

@Meisolsson sorry for not responding earlier, it was 6am my time and I forgot to respond :stuck_out_tongue_winking_eye:
Looks like you found the image, though I'd be open for another as well since I'm not sure GitHub even owns that image. It should be an official logo to avoid any copyright/licensing issues.
As for the username, they would be logged in at that point regardless right? I don't think they would need to worry about hiding/showing their username if they don't usually keep it public, as they've already logged in at this point.

The image should be replaced for copyright etc. But one with equal colors would be nice.
The problem about the e-mail is that the API only gives you the public email. So if you have not set a public email nothing will come up.

Ah interesting. Seems like an odd shortcoming of the API though. Best thing to do there IMO is just vertically center the username

Yeah had that thought too, will try it tomorrow.


fix non-pointer error in client get

@jdeppe-pivotal
sorry, I Fixed typo in Readme

Thanks for the fix!


Webpage notified when resource is still uploading

Current behavior : webpage is not updated until the shared file is completely uploaded.
Desired behavior : as soon as the QR-code is scanned, the webpage displays some "< connection established >" acknowledgement animated graphics, waiting for the actual resource to come soon.

9ac10c7b978bf2268d1d80314630076872e8ea78


Read Activity Data directly from InputStream

The input from the bot is marked as content-type "appliation/json; charset=utf-8"
even if the charset would not be explicitly stated, RFC8259 requires the
encoding to be UTF-8.
The approach to read individual bytes from the inputstream and treat
each byte as a character (ControllerBase.java) or even as url encoded
data (app.java) are just invalid and work by plain luck. If only ascii
input is tested, it most probably mostly works, but fails for all inputs
with high bytes set.
Jackson can decode JSON directly from an inputstream and then does the
right thing, there is no need to manually read the stream.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: matthiasblaesing sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


The Quotes API Fix

Implemented the Dynamic API specifications for the STR Quotes API. Which was statically pre-fetched by the next build command thus causing irregular behavior.

OK lets push to production


Added react resources

Fixes Issue
Closes #2137

Changes proposed

Added resource for React js

Screenshots

Note to reviewers

@rupali-codes  , Please review the pr and add relevant label.
This contribution is under hacktoberfest.

@Anmol-Baranwal @CBID2 , Please review the PR.
And let me know if any changes are expected.
And add relevant labels.

@Srish-ty.There seems to be a deployment failure.

I'd made changes in data.ts, probably that's why was getting error.
i've removed that part and now it's deployed successfully. commit link.
@CBID2 , please review the PR

Looks great to me! 😊 @Anmol-Baranwal, you're up! 😊

The issue is not assigned 😑 to him

@Srish-ty
Did you know that if we add a spam label, you won't be able to contribute to Hacktoberfest for a lifetime? This is because participants who are disqualified are not eligible to participate in the upcoming years.
At least comment for assigning the issue, how will someone know.

@Srish-ty Did you know that if we add a spam label, you won't be able to contribute to Hacktoberfest for a lifetime? This is because participants who are disqualified are not eligible to participate in the upcoming years.
At least comment for assigning the issue, how will someone know.

Sorry sir, but I didn't actually get you. To assign the issue to me?
Or to add issue to the pr.

@Srish-ty Did you know that if we add a spam label, you won't be able to contribute to Hacktoberfest for a lifetime? This is because participants who are disqualified are not eligible to participate in the upcoming years.
At least comment for assigning the issue, how will someone know.

Sorry sir, but I didn't actually get you. To assign the issue to me? Or to add issue to the pr. @Anmol-Baranwal , sorry if this has caused any frustration, but because of GitHub actions the comments in issues are locked and unable to comment 😕

Let me explain the process.
The issues are locked so that people don't tag us unnecessarily to assign the issue. We will always see it at our end and then we will assign the issues and unlock them if we need to clarify something or when we are assigning the issue.
You need to create an issue and allow us to assign you the issue before you start working and submit a PR. This is followed everywhere in Open Source.
This is outlined in the contributing guidelines.

You need to follow a predefined flow in order to contribute to Open Source. You may be new, so it is important to understand this. Most of the maintainers will mark it as spam and close it.
@CBID2 will tell you more about this.

@Srish-ty Did you know that if we add a spam label, you won't be able to contribute to Hacktoberfest for a lifetime? This is because participants who are disqualified are not eligible to participate in the upcoming years.
At least comment for assigning the issue, how will someone know.

Sorry sir, but I didn't actually get you. To assign the issue to me? Or to add issue to the pr. @Anmol-Baranwal , sorry if this has caused any frustration, but because of GitHub actions the comments in issues are locked and unable to comment 😕

Let me explain the process.
The issues are locked so that people don't tag us unnecessarily to assign the issue. We will always see it at our end and then we will assign the issues and unlock them if we need to clarify something or when we are assigning the issue.
You need to create an issue and allow us to assign you the issue before you start working and submit a PR. This is followed everywhere in Open Source.
This is outlined in the contributing guidelines.
You need to follow a predefined flow in order to contribute to Open Source. You may be new, so it is important to understand this. Most of the maintainers will mark it as spam and close it.
@CBID2 will tell you more about this.

@Anmol-Baranwal , oh okay, sorry for that.
Actually for few projects they asked to not wait till assigned n start working.
But again, sorry.

Creating a json file won't create the subcategory.
Follow contributing guidelines to add files.
Yeah in contributions.md , there are three types mentioned.
And I wanted to work on adding links/ resources part

@Anmol-Baranwal , oh okay, sorry for that. Actually for few projects they asked to not wait till assigned n start working. But again, sorry.

It's not your fault; this process is followed everywhere, from large organizational repositories (like Microsoft's) to smaller ones. Projects that don't follow this can lead to chaos. What if someone else makes a PR for the same issue, or if the PR doesn't align with the codebase, it may be closed, and that will reflect in your profile.
As I mentioned, maintainers of decent repositories will mark such PRs as spam. No worries, just make sure to read the contributing guidelines because it will always be mentioned.

@Anmol-Baranwal , oh okay, sorry for that. Actually for few projects they asked to not wait till assigned n start working. But again, sorry.

It's not your fault; this process is followed everywhere, from large organizational repositories (like Microsoft's) to smaller ones. Projects that don't follow this can lead to chaos. What if someone else makes a PR for the same issue, or if the PR doesn't align with the codebase, it may be closed, and that will reflect in your profile.
As I mentioned, maintainers of decent repositories will mark such PRs as spam. No worries, just make sure to read the contributing guidelines because it will always be mentioned.

@Srish-ty's issue has been approved and assigned @Anmol-Baranwal. Now you can review it! :)

@Srish-ty
I'm closing this issue due to inactivity. Please feel free to reopen it if you'd like to contribute or have further discussions.


Add notification and alert for Kviklet

Notification feature is necessary for "approval" application like Kviklet. And I think that it will be greater if Kviklet has notification/alert like Webhook or SMTP for approver and requester, i.e:

Requester raise a request -> Alert to Approver
Approver approved that request -> Alert to Requester
TemporarySession timeout -> Alert

Content in notification also needs to have some fields like "approver", "requester", "message", "request_url", "approve_url",...
Hope that it will be appear in next release, thank team!

Slack/teams and email notifications are definitely coming. But I don't think already in the next release sorry :( I think there is still some more pressing things to fix and do.
Would you mind sharing what notification integration you would need though? So I can at least prioritize the tooling you are personally using?

Thank for your update, I also think Slack/Teams is popular tool for notification integration and I'm using them ^^. Could I know which version the release air is?

I'd vote for Teams, if you're looking for popularity :)

I'll actually do this next week, I think it fits the theme of "support teams of 10+ people" that I am going for with v0.4 😁
It'll probably be basic and I don't want to link accounts yet but then you can at least have a teams channel linked where new requests and reviews are posted or something like that.

@VIDGuide @nh4ttruong I have merged #128 which allows you to configure a webhook URL.
New Requests that require review as well as new Reviews will then be alerted about in your configured channel.
I did not implement any kviklet-account to chat-app-user synchronization yet which would allow for more personalized notifications but I think this is a decent start.
If you have more specific wishes/requirements for your workflows, feel free to create a follow up ticket though! 😄

Btw it's kinda bullshit that Microsoft doesn't offer any sort of free tier for teams anymore. I had to sign up for a 30 day trial of Microsoft 365 but once that runs out I don't really have a way to test the integration anymore.
If anyone has worked with the teams webhooks before and has a good way of testing things, would be very happy about any tips.
If I don't touch it I'll assume it continues working, but especially to see how the message formatting actually  looks it would be good to have a way to actually try it out 😅


nodejs: fix impure configure phase

Description of changes
When running configurePhase from impure build environment (i.e. nix develop), default ./configure script was selecting wrong, impure Python installation instead of native build inputs.
https://github.com/nodejs/node/blob/d57af10d1be390f4e33e6efe596133de9da36c86/configure#L7-L15
Split from #262124
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[x] aarch64-linux → x86_64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@drupol, is this good for merge?


Sending form by clicking in a button

Hey, first of all. Congratulations for the only user friendly angularjs datetimepicker that doesn't require 1 billions clicks from the user to set the date/time.
I found an issue with ADM which is when the component is inside a form and I click in the "Today" button, as the button is not set as type="Button", the form is submitted.
It looks very easy to fix but I don't know if I have enough knowledge of angularjs and git to make a pull request of this.

Hi,
Thank you so much. glad to hear that :)
Yes! you are right!
Thanks for helping.
I'll fix that as soon as possible.


Ephemeris service

Change Description
The branch contains the full ephemeris service component. ephemeris_client.py controls the various interfaces. The outputs are uniformized. Each QueryResult returns a dataclass object or a QueryResult list.
Fixed issues:
--> closes https://github.com/lsst-sssc/ssoforcedphot/issues/6
--> closes https://github.com/lsst-sssc/ssoforcedphot/issues/7
--> closes https://github.com/lsst-sssc/ssoforcedphot/issues/8
--> closes https://github.com/lsst-sssc/ssoforcedphot/issues/10
New Feature Checklist

Now it is a complete ephemeris service. One can call the ephemeris_client.py with args and control the different interfaces, like JPL Horizons, Miriade, custom data loader
All of the relevant methods contains save_data options (default: False)

Documentation Change Checklist

Updated most of the docstring

I think data directory should go under tests

Discussion from the telecon is that ecsv handling is going to be updated

@talister  can you review if @akoumjian by your morning Tuesday?  It would be good to get this part into the repo


Parse file contents

Am I reading the instructions wrong, or are signatures on matching file name patterns.  Is there a reason the pattern matching of the content of files was not implemented?

It's by design you can see more here: https://github.com/michenriksen/gitrob/issues/54

Let me second the comments made in that post, thank you for all the great work.


simple GAN to generate mnist data :).

I have some progress in simple GAN with tinygrad.
fixin' batchnorm in personal fork.
@geohot suggestions

@kartik4949 Sounds really interesting! Will be looking into your fork once I figure out why tinygrad runs slower on GPU than on CPU 👍 Was also thinking about getting GANs working on tinygrad. Eventually a language model would also be amazing.

@skoshx sure , my guess is binary_op need optimization.

Done !!


Connection failure doesn't raise an exception

I'm running pytest against my server which happens to require a login.
As the connection didn't succeed I'd expect the async with AsyncMQTTClient() call to raise an appropriate exception instead of continuing.
-> async def _run_operation(self, operation: MQTTOperation[Any]) -> None:
(Pdb) b 481
Breakpoint 1 at /src/mqttproto/src/mqttproto/async_client.py:481
(Pdb) c

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB continue >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
> /src/mqttproto/src/mqttproto/async_client.py(481)_run_operation()
-> if operation.exception:
(Pdb) p operation.exception
None
(Pdb) p operation.response.reason_code
<ReasonCode.NOT_AUTHORIZED: 135>
(Pdb) c
………
    | Traceback (most recent call last):
    |   File "/src/mqttproto/tests/test_async_client.py", line 14, in test_publish_subscribe
    |     async with client.subscribe("test/+") as messages:
…
| mqttproto.MQTTProtocolError: cannot perform this operation in the DISCONNECTED state (must be in one of the following states: CONNECTED)

My goal was to provide transparent handling of intermittent issues, but maybe I went too far in trying to hide problems from the user. What would you suggest?


add the pip-install bin directory to path

In the old 7.X.X versions of the chart, we used to preprend the /home/airflow/.local/bin folder to the bash PATH.
We should consider what we can do to allow for this kind of usecase, as some people were depending on it, see here:
https://github.com/airflow-helm/charts/issues/163#issuecomment-823047250
NOTE: we no longer install into /home/airflow/.local/bin with pip --user but instead use pip --target /opt/python/site-packages, so we cant just go back to what we were previously doing

@simmartynas you were saying you used the old feature


Setting workbench.editorAssociations to change the default open mode of markdown to preview doesn't work

Type: Bug
Step 1: Set settings.json.
    "workbench.editorAssociations": {
        "*.md": "vscode.markdown.preview.editor"
    },

Step 2: Open any markdown file using vscode. I'm using the code --disable-extensions README.md command here.

Step 3: You can see that vscode doesn't open markdown as a preview.

VS Code version: Code 1.82.1<PHONE_NUMBER>d557a81c9d0b5f8a5a1e9274db5585, 2023-09-08T08:45:05.575Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 5 PRO 4650U with Radeon Graphics (12 x 2096)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.23GB (6.07GB free)

Process Argv
--disable-extensions README.md --crash-reporter-id f5996097-ae46-4ad3-8875-a294c592650a

Screen Reader
no

VM
0%

Extensions disabled
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
py29gd2263:30792226
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
3biah626:30602489
f6dab269:30613381
a9j8j154:30646983
showlangstatbar:30737416
962ge761:30835153
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
dsvsc015:30829745

I had this problem as well and just apparently found a fix but unsure why. I noticed on your setting, there is a comma after the closing bracket indicating there are other settings below it.
Mine was this way as well, but I removed it and went through the editor associations settings in the UI to add the key and it added it to the bottom of my settings - and now it works for some reason. So the fix for me was just to move it to the bottom. The only setting that was below it for me was "redhat.telemetry.enabled": false so maybe that particular setting messed it up for me but that was my experience?
Either way, here is my fully functioning settings.json for reference:
{
    "window.menuBarVisibility": "classic",
    "workbench.colorTheme": "Monokai",
    "workbench.startupEditor": "none",
    "redhat.telemetry.enabled": false,
    "workbench.editorAssociations": {
        "*.md": "vscode.markdown.preview.editor"
    }
}


Rework sanitization functions in m3u_tools

Currently the sanitize_attributes function in m3u_tools.IPTVChannelDoctor performs 2 sanitization operations at once (i.e. remove commas from attributes values and convert the attributes to lowercase).
There is one more sanitization, urlencode_logo, which is not performed as part of sanitize_attributes.
This is not homogeneous and shall be improved.
I like the idea of a function that performs all sanitizations at once, but users should also be given the freedom to choose which sanitizations they want to apply.
So said, this is a possible restructuring of this class:
urlencode_logo --> public function that url-encodes the tvg-logo attributes
attributes_to_lowercase --> public function that converts all attribute names to lowercase (e.g. tvg-ID to tvg-id)
convert_commas --> public function that converts all commas found in attributes values (except in tvg-logo) to underscores
sanitize_attributes --> public function that runs all sanitization functions on a channel's attributes

Please note: the functions in m3u_tools.M3UPlaylistDoctor shall reflect the structure above.

In the end, the proposed rework was improved.
Now the three classes in doctor offer a single public method, and this method is used to perform all sanitizations at once:
M3UDoctor.sanitize()
IPTVChannelDoctor.sanitize()
M3UPlaylistDoctor.sanitize()

With this approach, there's loss of granularity, but one usually doesn't know in advance when a playlist needs to be sanitized and what type of sanitization it needs. So, why not apply all of them regardless?


Does Norconex Http Collector(any version) support integration with Selenium for crawling dynamic web pages

Does Norconex Http Collector(any version) support integration with Selenium for crawling dynamic web pages (similar to Apache Nutch)?
We have dynamically generated web pages in our application where actual content of the page is loaded by javascript ajax calls after the webpage is loaded in browser. Default Norconex Http Collector is not able to crawl them since it doesn't really execute the javascript ajax calls (similar to the browser would do).

Hello,
I crawling pages with Dynamic content and looking to use the PhantomJS option, but the open source project has been closed for many years and not sure if some of the modern JavaScript extensions such as map would be supported.
Can I raise this as a feature request for support of a headless browser such as chrome or selenium

The integration of headless browser with Selenium is currently under development for the next major release (v 3.0). No release date can be estimated yet.

Implemented major browsers support in version 3 (headless).  It is also possible to configure when to use the browser vs. regular HTTP client based on URL patterns, and also have fallbacks if one fails.

Awesome


fix remove non-existant recognizer

When removing a recognizer, make sure it exists before removal. If recognizer has not been registered to manager inArray will return -1. The negative index passed to Array.splice will remove last element of the recognizers array.

@arschmitz :+1:

This looks good to go, thanks @ohnnyj !


An exception occurred while running Bloxstrap [BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
So I am using version 2.5.4 for the multi instance since the removal of it and I got this error code while trying to start Roblox "Response status code does not indicate success: 403 (Forbidden)" Anyone else got this?

same here

Might have to do with the fact that you're using a 9 month old version.


x/pkgsite: Error of the example of slog package

What is the URL of the page with the issue?
<EMAIL_ADDRESS>What is your user agent?
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Screenshot

What did you do?
Run the example of Example (LevelHandler)

Duplicate of #61885


Setting AutoPublishAlias value to a Ref reports E1019 violation on a different Property

*cfn-lint version: 0.21.4
Setting AutoPublishAlias value to a Ref, causes E1019 to be reported on a
different property that also uses that Ref as a part of Sub. This appears to be new to 0.21.4
as it is not present on 0.21.3; perhaps related to #582 and result of
d57c041181f761f98fe2e86fff4f709dd037db15 ?
Failing Example on 0.21.4 (note that FunctionName property is reported):
$ cat template.yaml
AWSTemplateFormatVersion: 2010-09-09
Transform: AWS::Serverless-2016-10-31
Parameters:
  EnvironmentName:
    Description: Environment where the function resides.

You should consider upgrading via the 'pip install --upgrade pip' command.
cfn-lint 0.21.4
E1019 Parameter EnvironmentName for Fn::Sub not found at Resources/Function/Properties/FunctionName/Fn::Sub
/var/tmp/template.yaml:11:3

$

No issue if a string value for AutoPublishAlias is used instead of a Ref:
$ cat template-string-alias.yaml
AWSTemplateFormatVersion: 2010-09-09
Transform: AWS::Serverless-2016-10-31
Parameters:
  EnvironmentName:
    Description: Environment where the function resides.

You should consider upgrading via the 'pip install --upgrade pip' command.
cfn-lint 0.21.4

$

Using a Ref for AutoPublishAlias does not report the error on 0.21.3. Our
use does include a default value for the Parameter which may be relevant to not
seeing the issue described in #582.
$ cat template.yaml
AWSTemplateFormatVersion: 2010-09-09
Transform: AWS::Serverless-2016-10-31
Parameters:
  EnvironmentName:
    Description: Environment where the function resides.

You should consider upgrading via the 'pip install --upgrade pip' command.
cfn-lint 0.21.3

$

I'm seeing the same issue. Setting the AutoPublishAlias with a !Ref throws an E1019 for every usage of !Sub that uses the referenced parameter.
An E1012 is also being thrown on the serverless function itself.
Seems to happen regardless of whether or not the referenced parameter has a default value set.

Yea, sorry about that.  I should have a fix in early next week with a release to get this fixed for you.


Re-add plugins

The current plugin architecture should be re-worked into a new form where we can integrate services into the PHPDI container, be able to add new Command line commands and even execute a little boostrapping code. Preferably this should be backwards compatible but I am not sure if that is possible.
Also: Investigate Puli

Closing the 3.0 tickets, will reconsider them after our re-alignment


Add typed-index feature

It allows user to replace Index with Index<T> when they enable typed-index feature.
Changes:

Added Index!(T) to lib.rs which conditionally refers to either Index or Index<T>
Index or Index<T> was moved to index.rs (for no particular reason)

Index or Index<T> is crated with crate::index::create_index which conditionally returns Index or Index<T>

I'm not sure if it's a good idea. Please do as you like :)

I just forked thunderdome and I don't need it actually.

It was a very bad idea, since feature flags are additive!


Every HLS Content on iOS thriows Error 4032 if preferNativeHls is enabled

Have you read the FAQ and checked for duplicate open issues?
yes
If the problem is related to FairPlay, have you read the tutorial?

not related
What version of Shaka Player are you using?

4.10.7
Can you reproduce the issue with our latest release version?
yes
Can you reproduce the issue with the latest code from main?
yes
Are you using the demo app or your own custom app?
both
If custom app, can you reproduce the issue using our demo app?
yes
What browser and OS are you using?
iOS 17.2 - 17.4
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
n/a
What are the manifest and license server URIs?

any HLS Content, also from the Shaka Demo Page
What configuration are you using?  What is the output of player.getConfiguration()?

i only configured: streaming.preferNativeHls=false;
What did you do?

followed the basic example, installed all Polyfills, added the config.streaming.preferNativeHls=false and started on iOS 17.2 and iOS 17.4.
What did you expect to happen?
Video would be able to play
What actually happened?

Error 4032 is emitted. This also happens on you official Demo Page, if I select any HLS Content.
If I use preferNativeHls=true, then Playback works absolutely fine.
Maybe I make a mistake and misunderstand something, but it feels very wrong, that a simple HLS Manifest cant be loaded when we use ManagedMediaSource (at least thats what I think is happening, when preferNativeHls is disabled).
Are you planning send a PR to fix it?

I'm not seeing any issues on my iPhone, playing HLS assets on the demo.
I'm using iOS 17.5.1, so it's possible there was a bug in the MediaSource implementation that was fixed sometime after 17.4.

I agree.
I tested it on iOS Simulators, and there, indeed nothing is working on any iOS Version (at least on my mac).
On real devices, uits working indeed.
That will make Development more complicated, but i guess, thats not really an issue then.
Thanks!

Oh yeah, more "obscure" APIs like MediaSource don't always work the same way in the iOS simulator as in an actual device. It's annoying, but there's not much you can do about it.


[WIP] Zone NTP settings only displayed when editing zone

https://bugzilla.redhat.com/show_bug.cgi?id=1095468

@miq-bot add-label ui, enhancement, wip

@miq-bot remove-label wip

@dclarizio - this is ready for review

@lgalis could you add some screen shots indicating the new information?  Thx, Dan

@lgalis Testing this out, I clicked on my default zone and got:
[----] I, [2015-09-30T14:45:21.501567 #2698:bb3988]  INFO -- : Started POST "/ops/tree_select/?id=z-1r1" for <IP_ADDRESS> at 2015-09-30 14:45:21 -0700
[----] I, [2015-09-30T14:45:21.557411 #2698:bb3988]  INFO -- : Processing by OpsController#tree_select as JS
[----] I, [2015-09-30T14:45:21.557626 #2698:bb3988]  INFO -- :   Parameters: {"id"=>"z-1r1"}
[----] I, [2015-09-30T14:45:21.899610 #2698:bb3988]  INFO -- :   Rendered ops/_settings_smartproxy_affinity_tab.html.haml (5.0ms)
[----] I, [2015-09-30T14:45:21.991154 #2698:bb3988]  INFO -- :   Rendered layouts/_flash_msg.html.haml (27.5ms)
[----] I, [2015-09-30T14:45:21.997312 #2698:bb3988]  INFO -- :   Rendered ops/_settings_evm_servers_tab.html.haml (94.6ms)
[----] I, [2015-09-30T14:45:21.997660 #2698:bb3988]  INFO -- :   Rendered ops/_all_tabs.html.haml (258.3ms)[----] F, [2015-09-30T14:45:21.998046 #2698:bb3988] FATAL -- : Error caught: [NoMethodError] undefined method `[]' for nil:NilClass
/home/dclarizio/dev/manageiq/app/views/ops/_settings_evm_servers_tab.html.haml:29:in `_app_views_ops__settings_evm_servers_tab_html_haml___3028126004390623279_139394060'
/home/dclarizio/.rvm/gems/ruby-2.2.3/gems/actionview-4.2.3/lib/action_view/template.rb:145:in `block in render'

Looking at my DB, that zone has it's settings field set to an empty hash.  Not sure how it got that way, but system is up and running, so perhaps we need to account for that.

@dclarizio - updated to account for no settings data


regression-test enable fault_injection_p0 cases

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


[Service Bus] Missing 'endpoint' error when endpoint has been provided

Package Name:  @azure/service-bus
Package Version: 1.1.6
Operating system: Windows
nodejs

version: 13.9.0

The bug is related to documentation in

source code documentation
SDK API docs on https://docs.microsoft.com

Describe the bug
The docs for creating the servicebus client with a token provider on microsoft.com seem to be out to date after looking in this repository. However, I followed the code in this repository and am still getting an error. I am trying to consume messages from a servicebus topic and this is the error:
TypeError: Missing 'endpoint' in configuration

After looking this error in the code in this repo I can't see why my code is giving this so I would appreciate some help. I am not sure if I have defined the token credential correctly but followed the code in this repo.
To Reproduce
Steps to reproduce the behavior:

Get your token you need to create the custom token credential.
Add it into the code below:

import * as azServiceBus from "@azure/service-bus";

var ServiceBusClient = azServiceBus.ServiceBusClient;

class CustomTokenCredential {
    getToken(scopes, options) {
      return new Promise(function (resolve, reject) {

          let accessToken = {
             token: <token>
             expiresOnTimestamp: <timestamp>
          }
          if (accessToken) {
              resolve(accessToken);
          }
          else {
              reject(Error("accessToken could not be resolved."));
          }
      });
  }
}

const topicName = <topicname>;
const subscriptionName = <subscriptionname>;
const host = "<namespace>.servicebus.windows.net";

async function main(){

  let customTokenProvider = new CustomTokenCredential();

  const sbClient = new ServiceBusClient(host, customTokenProvider);
  const receiver = sbClient.createReceiver(topicName, subscriptionName, "receiveAndDelete");

  try {
    const messages = await receiver.receiveBatch(10);
    console.log("Received messages:");
    console.log(messages.map(message => message.body));

    await receiver.close();
  } finally {
    await sbClient.close();
  }
}

main().catch((err) => {
  console.log("Error occurred: ", err);
});

Expected behavior
Expect to connect to servicebus and be able to receive messages. As I said above, the docs online do not seem to match with the code in this repo so it would be good to get some guidance on whether this is a bug or that my code is wrong.

Thanks for reporting @sydneyvert

The docs for creating the servicebus client with a token provider on microsoft.com seem to be out to date after looking in this repository.

Can you share a link to the docs you are referring to here?
They shouldnt be out of date and should match the 1.1.6 version of the @azure/service-bus package.
The master branch in this repository reflects the next version of @azure/service-bus package that we are working on and is still in preview.
We have tried to call this out in the our readme

Looking at the code you shared, you are using the sample code for the preview version in a project that is using the 1.1.6 version of the package.

Sorry my fault! I was originally following the createFromTokenProvider section here: https://docs.microsoft.com/en-us/javascript/api/@azure/service-bus/servicebusclient?view=azure-node-latest#createfromtokenprovider-string--tokenprovider--servicebusclientoptions-
Can you confirm if the above is what I should be following?
But that didn't seem to work either. Hence why I followed the code on here.

Here is my previous code, following the docs:
import * as azServiceBus from "@azure/service-bus";

var ServiceBusClient = azServiceBus.ServiceBusClient;
var ReceiveMode = azServiceBus.ReceiveMode;

class CustomTokenProvider {
    constructor(margin, valid) {
        this.tokenRenewalMarginInSeconds = margin;
        this.tokenValidTimeInSeconds = valid;
    }

    getToken(audience) {
        return new Promise(function (resolve, reject) {
            let tokenInfo = {
                tokenType: "jwt",
                token: "<token>",
                expiry: 3600
            };
            if (tokenInfo) {
                resolve(tokenInfo);
            } else {
                reject(Error("accessToken could not be resolved."));
            }
        });
    }
}

const topicName = "<topicname>";
const subscriptionName = "<subscriptionname>";
const host = "<namespace>.servicebus.windows.net";
const options = {};

async function main(){

    let customTokenProvider = new CustomTokenProvider(900, 3600);

    const sbClient = ServiceBusClient.createFromTokenProvider(host, customTokenProvider);
    const subscriptionClient = sbClient.createSubscriptionClient(topicName, subscriptionName);
    const receiver = subscriptionClient.createReceiver(ReceiveMode.receiveAndDelete);

    try {
        const messages = await receiver.receiveMessages(10);
        console.log("Received messages:");
        console.log(messages.map(message => message.body));

        await subscriptionClient.close();
    } finally {
        await sbClient.close();
    }
}

main().catch((err) => {
    console.log("Error occurred: ", err);
});

I get the error:
(node:16140) ExperimentalWarning: The ESM module loader is experimental.
Error occurred:  Error: connect ETIMEDOUT
    at TCPConnectWrap.afterConnect [as oncomplete] {
  name: 'ServiceUnavailableError',
  translated: true,
  retryable: true
}

I have doubled checked my proxy setting are correct. Please can you confirm if I am following the docs correctly? Thanks for your help.

Hey @sydneyvert,
Thanks for the sample code.

Please enable logs so that we can comprehend better about what's affecting

Preferably, set DEBUG environment variable to azure:service-bus to enable the logs at service-bus level.
If you're on windows, the command would be set DEBUG=azure:service-bus

Can you verify if the same sample code works fine if you use createFromConnectionString with the connection-string provided in the portal?

What was your motivation to use a custom token provider instead of using the login methods from  @azure/ms-rest-nodeauth?
[ Examples that leverage @azure/ms-rest-nodeauth for authentication - Azure account, interactive login or service principal ]

Regarding createFromTokenProvider - You're supposed to get the token by giving some sort of credentials in order to contact the service-bus namespace. I don't see that in your sample, would you let us know how you're getting the token.

My company have not permitted the use of SAS tokens and have blocked the connection string option so unfortunately this seems the only way.
I currently get the token using the msal package as this was recommended to my colleague by our microsoft contact.
The code below shows how I obtain the token using this method (this code runs in a browser):
 import * as Msal from "msal";

function loggerCallback(logLevel, message, containsPii) {
    console.log(message);
}

const config = {
    auth: {
        clientId: "<clientId>",
        authority: "https://login.windows.net/<tenantId>"
    },
    system: {
        logger: new Msal.Logger(
                           loggerCallback ,{
                                level: Msal.LogLevel.Verbose,
                                piiLoggingEnabled: false,
                                correlationId: '1234'
                           }
                   )
}
}

function authCallback(error, response) {
    console.log(error)
}

msalInstance.handleRedirectCallback(authCallback)

var scopes = ["https://servicebus.azure.net/.default"];

msalInstance.loginPopup(scopes).then(function (loginResponse) {
    //login success
    var idToken = loginResponse.idToken;
    console.log(idToken);
}).catch(function (error) {
    //login failure
    console.log(error);
});

if (msalInstance.getAccount()) {
    var tokenRequest = {
        scopes: ["https://servicebus.azure.net/.default"]
    };
    msalInstance.acquireTokenSilent(tokenRequest).then(response => {
        // Acquire token silent success
        // Call API with token
        var accessToken = response.accessToken;
        console.log(accessToken);
    }).catch(error => {
        //Acquire token silent failure, and send an interactive request
        if (error.name === "InteractionRequiredAuthError") {
            return msalInstance.acquireTokenPopup(tokenRequest)
                .then(response => {
                    var accessToken = response.accessToken;
                    console.log(accessToken);
                }).catch(function(error) {
                // Acquire token interactive failure
                console.log(error);
            });
        }
        console.log(error);
    })

I followed set DEBUG=azure-servicebus and got nothing further. Just the same timeout error. Is the AMQP web socket set by default?

I followed set DEBUG=azure-servicebus and got nothing further.

Our apologies @sydneyvert
The DEBUG env variables should be set to azure:service-bus not azure-servicebus. Can you try again with the new value?

Is the AMQP web socket set by default

Web sockets are not enabled by default. I see that you mentioned proxy in one of your comments above. You can refer to the useProxy.ts sample to see how to use web sockets.
Are you running your application in the browser or node? If you are using the browser, then you can use the built-in WebSocket instead of depending on the ws package.

I missed your earlier statement that your code runs in the browser.
In that case, yes, web sockets are used by default.
But if you are behind a proxy, then you need to use the sample I linked above to pass in the proxy settings. Since the code runs in the browser, you can use window.WebSocket instead of the WebSocket from the ws package.
Also, instead of msal, you can use the @azure/ms-rest-browserauth package. You can pass the credentials you get from using this package to the createFromAadTokenCredentials() function to create the ServiceBusClient.
Please do try with the above and share the logs if you still have issues

Hello @sydneyvert,
At first glance, a couple of updates to the msal-sample-code that you've provided.

You're passing the scopes array to the .loginPopup() method directly which is invalid since the method expects a bag of options(AuthenticationParameters) among which the scopes in one of the attributes.

type AuthenticationParameters = {
    scopes?: Array<string>;
    extraScopesToConsent?: Array<string>;
    prompt?: string;
    extraQueryParameters?: StringDict;
    .
    .
    .
}

So, please update that .loginPopup(scopes) call appropriately.
For example - .loginPopup( { scopes: scopes } ).

Update the scope for service-bus to https://servicebus.azure.net//user_impersonation

Thanks @HarshaNalluru, I have made those changes.
@ramya-rao-a thanks for your quick replies. The login script runs in the browser, however the service bus part actually runs in node. Can I still use the example above to allow the use of AMQP web socket?
I have set DEBUG=azure:service-bus and am not getting any more information from my logs apart from the ServiceUnavailableError

@ramya-rao-a used the web socket example you suggested and it all works perfectly now. Thanks so much for all of your help.

That is great to hear @sydneyvert
@HarshaNalluru, At the very least we should make the below changes:

Update jsdocs for createFromTokenProvider to mention that if one is using their own token provider against AAD, then what should be the "scopes" value to use.
Do the same in the readme under the "Authenticate the client" section
Update useProxy sample files that if in browser one does not need to pass in a WebSocket from the ws package

The latest update to the @azure/service-bus package with version 1.1.7 has the above doc changes


Cannot read property 'contains' of null

This error occurs when I click "outside" of the component which is not rendered yet.
So the onOutsideEvent is triggered even if the component is not visible.
See the search widget from this page: http://www.amihaiemil.com
The component which should disappear on outside click is the list of results. Type "test".
The error appears on the page at every click.

A PR is welcome.

Right; I'll try to look into it :D

@gajus I also deployed and it's working fine, thanks. Closing this :)


On State hover, how to show some description text about the state?

I want show some description about the State on mousehover. So far i could not find any option to do this. Kindly help me out.

Since the state name comes from data.name you could create a list of items to get for each state depending on that abbreviation (e.g.,  $('#tooltip').text(informationList[data.name]).show();)


DCR software_statement missing from response

Retrieving a registration using the conformance tool is failing due to the software_statement field being missing from the response. According to the spec below, the software statement must be returned if one was used as part of registration.
OAuth 2.0 Dynamic Client Registration Protocol: https://www.rfc-editor.org/rfc/rfc7591#section-3.2.1

If a software statement was used as part of the registration, its
value MUST be returned unmodified in the response along with other
metadata using the "software_statement" member name

Background to this issue:
There is a bug in the OB Directory representation of an SSA, the software_version field should be a string but in the OB SSA it is a number. This causes software statement validation to fail in AM, as it inspects and validates the statement.
Therefore, we do not enforce that a software_statement is required for DCR in AM OAuth2 config. In the register request in the OB API, we null off the software_statement field, so that it is not passed down to AM (to avoid this bug).
This means that we have to do validation of the software statement in IG (the SSAVerifier filter).
If OB ever fix the issue then would need to make the following changes:

Fix the ProcessRegistration groovy to not null off the software_statement field
Configure an OAuth2 Software Publisher for the OpenBanking Ltd issuer
Add the OB certs to the am truststore, see https://backstage.forgerock.com/knowledge/kb/article/a46349648

The workaround to fix this issue is to store the SSA in IDM as part of the apiClient object. The apiClient.ssa currently exists for this purpose, but it is not being populated due to a bug.
Then for GET registration requests we can retrieve the SSA from IDM. For PUT/POST requests, then we should play back the ssa that was sent in the request as part of the response.

We should also review the behaviour of modifying a registration. There is currently no handling of updates in the CreateApiClient filter.
Review handling of updating registrations, implement update logic to CreateApiClient


run on corol tpu

After converting the model to tflite format. Is it possible to use it with corol tpu ?

sorry, i does not try this. I think this is depend on tflite.


Add buy market order choice

Add the possibility to choose between the buy limit order or the buy market order
Idea: --price with an empty value could mean a market order for instance ?

Or maybe --market ?

Or maybe --market ?

Definitely better 👍

Definitely better 👍

The dev branch has been updated to support the market buy order using --buy_type market. All the information or in the README of the same branch.


Incorrect function name in output from flutter driver

Currently printing " To enable the driver extension call registerFlutterDriverExtension first"
Expected to print " To enable the driver extension call enableFlutterDriverExtension() first" and also tell me what library to import.
Or, just point the user to a URL that has all the good stuff.

We probably won't need to do this, since we plan to generate a new test_driver/foo/main.dart that connects your main app to enableFlutterDriverExtension.


file.trimTrailingWhitespace should have a per-language configuration

Hi, I have turned this option on, but I find it a little difficult to work with git add -p 's "manually edit hunk".
This opens a file addp-hunk-edit.diff in Code and this is full of trailing whitespaces that must be preserved. Otherwise, git will not function correctly. So I have to turn this option off every time I want to manually edit a hunk.
I think it would be more convenient if this option can be turned on or off depending on the language mode or filename. For example, I can add a rule that says *.diff files should not have trailing whitespaces trimmed while *.cpp files should.
Thanks!

   "[diff]": {
     "files.trimTrailingWhitespace": false
    },

And then turn it on for all other files with the main setting.

Thanks!


.nav-justified class is missing

see also: http://stackoverflow.com/questions/34411884/justify-nav-pills-with-bootstrap-v4/34433022. Should it be add?

I presume you mean .nav-justified. Approximate duplicate of #17723.


Use afterMinimumDuration to correct frame pacing

The current nano sleep based frame pacing does not work well on Mac, and leads to inconsistent frame pacing. 30 FPS looks very stuttery. With this change, the frame pacing is perfectly showing a solid blue line on Metal HUD.

Remember to check power consumption before and after to avoid regressions. This should ideally be done in two cases:

Empty 2D project
Complex 3D scene (but where the FPS cap is consistently reached)

On a laptop, you can measure battery life, or how much time it takes for the battery to drop by say, 10%.

@mrsaturnsan we may need to verify that Godot is not adding its own delays or doing its own frame pacing on top of that

There is a delay in the main loop, so I wonder if that is interfering with your pacing?

Thanks!


[Storage] Add overall crc64 calculation to partitioned upload for all packages

This change adds the calculation and check for an overall, cumulative crc during partitioned upload for all packages. Basically, it maintains two things, an overall crc while reading from the input stream and the individual crcs for each chunk. Once all chunks are uploaded, the list of chunks is sorted and all the chunk crc values are concatenated together. This can then be compared to the overall crc from the input stream. This can potentially catch issues in the internal upload code with regards to chunk partitioning or ordering.
This required some refactoring to how information about the chunks are stored and returned. Some changes were to existing logic before content validation to make the logic common across different types of uploads. Also sprinkled in some type hints to make the file a little easier to work with/understand.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-blob


Missing properties: minecraft:behavior.defend_trusted_target

Missing documentation on the following properties:

[ ] on_defend_start
[ ] sound_chance

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1c5dce6b-8286-7980-d240-9050f91ae35b
Version Independent ID: fb48dc2c-9528-c3d7-6092-87a0cd69bd46
Content: minecraft:behavior.defend_trusted_target
Content Source: creator/Reference/Content/EntityReference/Examples/EntityGoals/minecraftBehavior_defend_trusted_target.md
Product: gaming
GitHub Login: @v-josjones
Microsoft Alias: v-josjones

This has been updated  Thanks for submitting.


Role required to be added to Weekly Digest

Please document the minimum permissions / roles required to receive the weekly digest.  Currently I cannot "Include" AAD users or guests to receive these emails.  We have third party auditors and systems that need these emails and I currently cannot "Include" them

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f5a932ce-6046-f6a9-9b85-188c8530174c
Version Independent ID: 196cb921-fbf8-6819-e0e2-8cd03fda4505
Content: Email notifications in PIM - Azure Active Directory
Content Source: articles/active-directory/privileged-identity-management/pim-email-notifications.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@Torxsmind Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hi @Torxsmind
Thanks for the feedback! I think it is documented: "A weekly Privileged Identity Management summary email for Azure AD roles is sent to Privileged Role Administrators, Security Administrators, and Global Administrators that have enabled Privileged Identity Management...It is only available for Azure AD organizations on the public cloud."
#please-close


updated to support multi-project repos

closes #61
add dynverTagPrefix that is empty by default but override with name.value of the project to support tags like -v
mostly inspired by #62

seems like it failed some binary compat check...
[error] sbt-dynver: Failed binary compatibility check against com.dwijnand:sbt-dynver:4.0.0 (e:sbtVersion=0.13, e:scalaVersion=2.10)! Found 3 potential problems (filtered 5)
[error]  * method this(scala.Option,java.lang.String,Boolean)Unit in class sbtdynver.DynVer does not have a correspondent in current version
[error]    filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("sbtdynver.DynVer.this")
[error]  * method this(Boolean)Unit in class sbtdynver.GitDescribeOutput#Parser does not have a correspondent in current version
[error]    filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("sbtdynver.GitDescribeOutput#Parser.this")
[error]  * method apply(scala.Option,java.lang.String,Boolean)sbtdynver.DynVer in object sbtdynver.DynVer does not have a correspondent in current version
[error]    filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("sbtdynver.DynVer.apply")

how to deal with this?

Duplicate of #158


OAEP encryption using SHA1?

I've posted this issue on the public-encrypt repo as well:
https://github.com/crypto-browserify/publicEncrypt/issues/13
Public Encrypt is using sha1 for the hash function, is that secure? It sounds like sha1 is being deprecated because it's been theoretically crackable since 2005. Is this the same hash function that node crypto is using / are they not on sha2 / sha256?

yes probably but the goal here is to be fully compatible with the node crypto api, so we'll remove this when they do.


Improve GPU deployment experience

After almost 2 months since the GPU mainnet launch, we've gathered many feedbacks from the community, both from the crypto and non-crypto native users. What comes out the most is the ease of use to deploy, a "point and click solution".
Hence, I created this topic to track the advancement on a new type of form similar to the SDL builder, but verticaly aligned for GPUs.
The form will support:

Fully abstracted SDL
Simplified options
Template selection for GPU related deployments
Easy token selection
Easy region selection with provider attributes

Once https://github.com/akash-network/community/issues/344 and https://github.com/akash-network/support/issues/124, further improvements will be enabled, making the experience of deploying GPUs better than ever!


Trafficcontroller Spec Typo ?

Hi,
I've got a question.
This line looks like a typo to me, is that the case or am I wrong?
https://github.com/cloudfoundry/loggregator-release/blob/82e93bab660bde17d9bf175e5927b06f4e58d65c/jobs/loggregator_trafficcontroller/spec#L70
Regards,
Benjamin

@benjaminguttmann-avtq Thanks for opening an issue! I think you might be referring to addrs, which is just the pluralization of addr, an abbreviation for address. You can see it used throughout this repo - here's an example:
https://github.com/cloudfoundry/loggregator-release/blob/ac44ccc81f8b41b064a6ea452628cdf46c150872/jobs/reverse_log_proxy/templates/environment.sh.erb#L8
So, I think the line appears as intended. If I misunderstood your intent, feel free to comment on this issue again and we can reopen it.


Use a date prefix on the s3 bucket

When we change buckets for storage, we have to coordinate the changes across a few different places. It would be a little bit easier if the bucket always stayed the same, and the data was just put into date prefixes.

Which date?
Might this be something where the version number (of the software / queries) would be helpful?

I was thinking date of deployment, which is how we name the buckets now. But it could be any version number.
This is the sort of thing that would change less frequently than the software/query changes; we really only change it when we don't want to show stale data in dev, or when we need to atomically switch the tilesets in prod.

Additionally, we'll want to consider following s3 best practices for our object key names. This effectively boils down to adding a hash prefix to the name. Thanks @zerebubuth!
http://docs.aws.amazon.com/AmazonS3/latest/dev/request-rate-perf-considerations.html
For this to work though, we'll need to ensure that fastly can manipulate the urls accordingly. Some research needs to be done here to verify, although at first glance these functions look promising:
https://docs.fastly.com/guides/vcl/cryptographic-and-hashing-related-vcl-functions

@rmarianski, how should we QA this one?

The tiles should still be loading at the same rate. On the back end the s3 bucket will have objects sitting in a different location.

TileBRB: This is deployed in dev. That everything is still working via S3 means this is working?

With today's new dev deploy, I see a new dir in the dev S3 bucket with today’s date on it, and the dated dir is there there, and no new garbled root dirs so that verifies this issue.


Automate API doc generation of methods

Issue is #215. Here are the changes:

When a new method is added, Sphinx doesn't have to be told about it, because we pass some hints to autoclass, like so:

.. autoclass:: fabrictestbed_extensions.fablib.fablib.FablibManager
   :members:
   :special-members:

This will ensure that Sphinx will automatically include public methods and special methods (such as __init__ and __str__) in the generated docs, without having to explicitly add them.

Each of the fablib modules get their own HTML file, instead of one giant fablib.html.

Toctree has been updated/included accordingly. Thus we get a left-hand side navigation bar, as seen in the docs buid of this PR.

Added a docs/source/_static/.empty so that Sphinx won't complain about this directory not being present:

https://github.com/fabric-testbed/fabrictestbed-extensions/blob/68ba4ae4cea0116c9d757357c594039998e8e689/docs/source/conf.py#L57

A tox docs environment has been added, so that tox -e docs will generate API docs locally.
README has been updated accordingly.

Perhaps we can delete sphinx.sh now?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 20.726%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
952

Relevant Lines:
4101

💛  - Coveralls

Removed sphinx.sh after speaking with Paul.


Code Signing problem

New Issue Checklist

[X] I searched for existing GitHub issues
[X] Updated fastlane to the latest version
[X] I read docs.fastlane.tools
[X] I read the Contribution Guidelines

Issue Description
I am getting a signing error and I don't know what to do.. After hours of searching, reinstalling certificates and everything. It still won't work. This error is also not listed on: https://docs.fastlane.tools/codesigning/common-issues/
Does anyone know a solution?
Complete output when running fastlane, including the stack trace and command used

The following build commands failed:
Check dependencies
(1 failure)
[15:58:30]: Exit status: 65
[15:58:30]:
[15:58:30]: Maybe the error shown is caused by using the wrong version of Xcode
[15:58:30]: Found multiple versions of Xcode in '/Applications/'
[15:58:30]: Make sure you selected the right version for your project
[15:58:30]: This build process was executed using '/Applications/Xcode.app'
[15:58:30]: If you want to update your Xcode path, either
[15:58:30]:
[15:58:30]: - Specify the Xcode version in your Fastfile
[15:58:30]: ▸ xcversion(version: "8.1") # Selects Xcode 8.1.0
[15:58:30]:
[15:58:30]: - Specify an absolute path to your Xcode installation in your Fastfile
[15:58:30]: ▸ xcode_select "/Applications/Xcode8.app"
[15:58:30]:
[15:58:30]: - Manually update the path using
[15:58:30]: ▸ sudo xcode-select -s /Applications/Xcode.app
[15:58:30]:
+---------------+-------------------------+
|            Build environment            |
+---------------+-------------------------+
| xcode_path    | /Applications/Xcode.app |
| gym_version   | 2.64.1                  |
| export_method | app-store               |
| sdk           | iPhoneOS11.1.sdk        |
+---------------+-------------------------+
[15:58:30]: ▸ === BUILD TARGET X OF PROJECT X WITH CONFIGURATION Release ===
[15:58:30]: ▸ Check dependencies
[15:58:30]: ▸ Code Signing Error: Provisioning profile "X" doesn't include signing certificate "Y".
[15:58:30]: ▸ Code Signing Error: Code signing is required for product type 'Application' in SDK 'iOS 11.1'
Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.1

Ruby
2.2.4

Bundler?
false

Git
git version 2.13.6 (Apple Git-96)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.13.1 (17B48)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
true

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
`./fastlane/Fastfile`
# Customize this file, documentation can be found here:
# https://docs.fastlane.tools/actions/
# All available actions: https://docs.fastlane.tools/actions
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "2.64.1"

default_platform :ios

platform :ios do
  before_all do

    cocoapods

  end

  lane :store do
    cert
    sigh
    pem
    # build your iOS app
    gym(
      scheme: "x",
      workspace: "x",
      export_method: "app-store",
      clean: true,
      include_bitcode: true
    )
    #appstore
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful
  end

  error do |lane, exception|

  end
end

# More information about multiple platforms in fastlane: https://docs.fastlane.tools/advanced/#control-configuration-by-lane-and-by-platform
# All available actions: https://docs.fastlane.tools/actions

# fastlane reports which actions are used. No personal data is recorded.
# Learn more at https://docs.fastlane.tools/#metrics

`./fastlane/Appfile`
app_identifier "x" # The bundle identifier of your app
apple_id "x" # Your Apple email address

team_id "x" # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.64.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

CFPropertyList
2.3.5

activesupport
4.2.10

addressable
2.5.1

babosa
1.0.2

bigdecimal
1.2.6

bundler
1.14.6

claide
1.0.2

cocoapods
1.3.1

cocoapods-core
1.3.1

cocoapods-deintegrate
1.0.1

cocoapods-downloader
1.1.3

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

cocoapods-trunk
1.3.0

cocoapods-try
1.1.0

colored
1.2

colored2
3.1.2

commander-fastlane
4.4.5

concurrent-ruby
1.0.5

declarative
0.0.10

declarative-option
0.1.0

domain_name
0.5.20170404

dotenv
2.2.0

escape
0.0.4

excon
0.55.0

faraday
0.12.1

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.1.0

fourflusher
2.0.1

fuzzy_match
2.0.4

gh_inspector
1.0.3

google-api-client
0.13.6

googleauth
0.5.1

highline
1.7.8

http-cookie
1.0.3

httpclient
2.8.3

hurley
0.2

i18n
0.9.1

io-console
0.4.3

json
1.8.1

jwt
1.5.6

libxml-ruby
3.0.0

little-plugger
1.1.4

logging
2.2.2

memoist
0.15.0

mime-types
3.1

mime-types-data
3.2016.0521

mini_magick
4.5.1

minitest
5.4.3

molinillo
0.5.7

multi_json
1.12.1

multi_xml
0.6.0

multipart-post
2.0.0

nanaimo
0.2.3

nap
1.1.0

netrc
0.11.0

nokogiri
1.5.6

os
0.9.6

plist
3.2.0

power_assert
0.2.2

psych
2.0.8

public_suffix
2.0.5

rake
10.4.2

rdoc
4.2.0

representable
3.0.4

retriable
2.1.0

rouge
1.11.1

ruby-macho
1.1.0

rubygems-update
2.5.2

rubyzip
1.2.1

security
0.1.3

signet
0.7.3

slack-notifier
1.5.1

sqlite3
1.3.13

terminal-notifier
1.7.1

terminal-table
1.7.3

test-unit
3.0.8

thread_safe
0.3.6

tty-screen
0.5.0

tzinfo
1.2.4

uber
0.0.15

unf
0.1.4

unf_ext
<IP_ADDRESS>

unicode-display_width
1.1.3

word_wrap
1.0.0

xcodeproj
1.5.3

xcpretty
0.2.6

xcpretty-travis-formatter
0.0.4

generated on: 2017-11-16

Facing the same issue.
Even if I have defined the export_options.provisioningProfiles to match the app ids with provisioning profiles, it always picks one provisioning profile and tries to sign with it.

Right now im bypassing this by setting the provisioning profile manually using update_project_provisioning action

After 3 full days of searching... It seemed the certificate from my company was not completely right. After deleting every certificate under login I got the right certificate. In Xcode I also had the wrong profision profile. I had selected "Project Name" but I needed "Project Name Appstore ". Hope this will point anyone in the right direction.


PropertyPathEndpoint ignores port when it compute serviceName

Hi,
I'm testing spring-cloud-conifg with spring-cloud-bus.
I'm using gitlab repo. So I set my webhook.
My configserver instance received the hook and send an event in my amqp queue. The message is received by my config client instance but it does nothing.
Here is the amqp message received by the config client instance :
2016-06-29 18:56:21.416 DEBUG [springCloudBus.anonymous.rY9eo0eZQ-6VOphA8MCSGA-1] BlockingQueueConsumer - Received message: (Body:'{"type":"RefreshRemoteApplicationEvent","timestamp":1467219381398,"originService":"configServer:tbd:8080","destinationService":"myapp:dev","id":"c56c0148-46af-4105-968d-d087748b0621"}'MessageProperties [headers={contentType=text/plain, originalContentType=application/json}, timestamp=null, messageId=null, userId=null, appId=null, clusterId=null, type=null, correlationId=null, replyTo=null, contentType=text/plain, contentEncoding=null, contentLength=0, deliveryMode=null, expiration=null, priority=0, redelivered=false, receivedExchange=springCloudBus, receivedRoutingKey=springCloudBus, deliveryTag=1, messageCount=0])

Am I missing something ?
Thanks

Any more information you could give would be helpful, versions, configuration, etc...

Here are the different information
my client configuration bootstrap.yml
spring:
  application:
    name: myapp
  rabbitmq:
    host: <rabbitmq_host>
    port: 5672
    username: <username>
    password: <password>
  cloud:
    config:
       uri: <configserver_uri>
       failFast: true # echec demarrage si config server injoignable
       label: feature-piranas-support #tant qu'on est sur la branche.  I can see the message consumed  by my client. What is the classe which trigger the rebinding after received amqp message ?
Thanks

I guess I'm not understand at well the flow.
I thought when my got repo execute the webhook /monitor on my config server instance , this one emit an event each application receives. If one instance is concerned by the change detection it reload its application's configuration.
I made a second test by disabling /monitor webhook and calling on one of the instance the /bus/refresh?destination="myapp:**". So far so good, the correspond instances receives the event and reload their configuration.
Can you details the flow. What is the benefit of webhook if I have to call /resresh endpoint ?

Sorry,
I found the problem : I simulated webhook with a modified attribute like <application_name>-.yml. So my client received  the event but ignored it.
curl -H "Content-type: application/json" -H "X-Gitlab-Event: Push Hook" -d '{"commits": [{"id": "b6568db1bc1dcd7f8b4d5a946b0b91","message": "Update application configuration.","timestamp": "2011-12-12T14:27:31+02:00","author": {"name": "Me","email": "me@mymail.fr"},"added": [],"modified": ["conf/<application_name>-.yml"],"removed": []}]}' -X POST http://<config_server_uri>/monitor -vvv
If I changed my request body by a modified attribute like <application_name>.yml it works.
curl -H "Content-type: application/json" -H "X-Gitlab-Event: Push Hook" -d '{"commits": [{"id": "b6568db1bc1dcd7f8b4d5a946b0b91","message": "Update application configuration.","timestamp": "2011-12-12T14:27:31+02:00","author": {"name": "Me","email": "me@mymail.fr"},"added": [],"modified": ["conf/<application_name>.yml"],"removed": []}]}' -X POST http://<config_server_uri>/monitor -vvv
Thanks

Finally I guess there's an issue about the computation of serviceName in the method guessServiceName of the PropertyPathEndpoint.
If I have a file named <my_application>-<my_profile>.yml, the component build 2 destinationServices :

"<my_application>:<my_profile>"
"<my_application>-<my_profile>:**"

Unfortunately, the default appId computation as mentionned spring-cloud-bus documentation combines <my_application>:<my_profile>:<my_port>
So I think the component should append "<my_application>:<my_profile>" with ** like that "<my_application>:<my_profile>:**" to correctly addressed the targeted application.

@RaphC PR?

The problem is actually better fixed in the bus IMO (see link above).

Hy dsyer,
Thank you.


Updated README and renamed some files, no source-code updates excluding some commentary in functions.py

Wait, there's only 1 file changed


Fix #98 - drop python2.6 support

Coverage remained the same at 77.049% when pulling 417d6b1f158e23450a2c507c1d18ddcd5e009421 on diraol:fix-98-drop-python2.6-support into 5ec2927f332708c76103bf199d05094c8a92badf on klen:develop.


Support ScanContext for Disaggregated Read

What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tiflash/issues/6827
Problem Summary:
What is changed and how it works?
Make explain analyze work for Disaggregated read.
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge


del user and role after disable auth

After disable auth,use the non-auth client connection to delete test user and role.

@stonelion have you manually test this change?

This test wasn't failing in CI (see https://travis-ci.org/coreos/jetcd/builds/231622081) -- is CI broken or is the test OK as-is?
Why did the file permissions change from 644 to 755?

@heyitsanthony windows change the file permissions automatically??

@fanminshi yes.

@fanminshi this pr makes sense. can we cheery pick this and get it merged?
or @stonelion can you rebase with master?

@xiang90 I'll cherry pick the commit.

close via https://github.com/coreos/jetcd/pull/103

@fanminshi thank you


use ISO 3166-1 Alpha-2 codes to identify nations

Currently, we identify nations in the folder structure and in the API /data/nation/{key} path by the uppercase English display name of the region. I have verified that both javascript and PHP use the same display names for nations, so it's not too much of a problem.
However, it's not so simple to derive the ISO code from the display name of the nation, especially once we've transformed it to uppercase. It's almost impossible to go from uppercase to the original display form. So there is a little bit of overhead in keeping for example a country_iso property in the /calendars metadata endpoint's national_calendars data,  or in creating frontend map structures to derive the ISO code from the nation name. If we just used the ISO code to store data about nations, we wouldn't need frontend map structures, since it's easy enough to derive the display name from the ISO code. This will also provide for better folder structures, avoiding strange folder names with spaces and ampersands and whatnot.

Fixed in commits:

c4eb794663acfa0cc988597de8a3c1fb3724cd4d
3d3a0291b845eceecff9b0a5778ce88a8d6ad9bb
2a5561ca0aa09c1e38fd57a3da7c7b5c010983a8

Perhaps also some fixes were mixed into a few other commits here and there, if something was missed in these three commits...
All seems to be working fine so closing as completed.


Networking RPC Error

I'm tagging @Faless  and @Calinou since they have helped a lot of other people in things like this.
Godot version:

Godot Version 3.2
OS/device including version:

Issue description:

I was trying to start a game using an RPC. Instead, I got an error:
ERROR: Node not found: Host.    At: scene/main/node.cpp:1381 ERROR: _process_get_node: Failed to get path from RPC: Host.    At: core/io/multiplayer_api.cpp:256 ERROR: Invalid packet received. Requested node was not found.    At: core/io/multiplayer_api.cpp:204
Steps to reproduce:
I made a menu with a join and host button. Hosting would bring you to a scene where you would be given a port number, and joining would show you a spin box where you would enter in the port. Everything is connected, but when the host starts the game, I got the error.
Minimal reproduction project:

GitHelp.zip
Thanks for any help!

Please don't ask support questions here, as this issue tracker is meant to be used for bug reports only. Use one of the other community channels instead.

Thanks, sorry, I'll ask on the godot forum or something next time.


Commit algebra

This is a very nice feature that git has that defines a language for specifying commits. Things that might be nice to borrow include:

foo^ foo's parent
foo~2 foo's parent's parent
foo..bar commits between foo and bar

we have support for ^ and ~
Closing this. If there are other requests, they can be filed as new issues.


README: Made some changes

Made changes to the README for better readability.

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Update to Kafka 0.9

Kafka 0.9 is now the latest, any chance for an update soon?

I haven't had time to look into the new API yet, I'll try in the next days - haven't really had much spare time recently. Thanks for asking!

Version 0.4.1-kakfa09 supports Kafka 0.9.


Refactor to use BP stack instead of rust-bitcoin

This makes library much more lightweight, helps to stabilize API and avoid major refactorings with each new rust-bitcoin release.

Sorry, GitHub opened PR in the wrong repo! It was intended to be against the fork In https://github.com/BP-WG/bp-esplora-client


Use label for Input title and subtitle

There are two ways to pair a label and an input:
<!-- IMPLICIT LABEL (not recommended for use) - the label wraps the input. -->
<label>
  Name:
  <input type="text" name="name" />
</label>

<!-- EXPLICIT LABEL (recommended for use) - the label is connected to an input via "for" and "id" -->
<label for="explicit-label-name">Name: </label>
<input type="text" id="explicit-label-name" name="name" />

Realworld usage

Sometimes we use only a subtitle.
"Forgot password" link is part of the subtitle. (Should be refactored. <label> should not contain interactive elements)

Multiple labels issue
Multiple label elements can be given the same value for their for attribute; doing so causes the associated form control (the form control that for value references) to have multiple labels.
From the specification:

The LABEL element may be used to attach information to controls. Each LABEL element is associated with exactly one form control.

Creating multiple labels linked to the same input can cause an issue with screen readers.

Basically, I haven't seen any subtitles in input lables ever. Usually it works like this: there is a (i) button with tooltip of additional info

@vozbrann What's the status?


Add a brown palette

from https://twitter.com/jobosonchisa/status/1587825975684890626?s=20&t=qHNaa3oE6M_cg2s46p0eBg

or consider moving from open-color to colar because it's done all the work i was planning on doing https://github.com/fchristant/colar

Hey @argyleink,
I think switching to colar is a great idea!
Love the "feel" of the new palette.
It would also address the issue of the current palette's limited choice of dark colors #247
I'd be happy to help with the migration.
Thank you for maintaining this awesome project!

fixed in 1.5