Not Listening ver 10.0.19041.1202  x86

I dont know why the prev. issue was closed!!!
x86 ist still missed.

Same issue after latest update.  I removed the update and working again.  Additionally I paused updates until October 21 (which is the max allowed at the time).   Hopefully there is a fix.

installing autoupdater downloaded a new INI that supports 1202 but I cant get it to work. I swear I tried everything.

https://github.com/asmtron/rdpwrap/blob/master/binary-download.md

I rolled back the update, but no way. It still shows 1081 as fully supported but nothing. it just does not work :(
when connecting from another PC it disconnects local session, and rdpcheck just switchs users.

installing autoupdater downloaded a new INI that supports 1202 but I cant get it to work. I swear I tried everything.
https://github.com/asmtron/rdpwrap/blob/master/binary-download.md

Autoupdate does not concider x86 because the author only updates x64

I use x64
Ariel Fernández
Director
[AFER IT & Internet Solutions]
El vie., 17 de septiembre de 2021 20:03, IljaM13 @.***>
escribió:

installing autoupdater downloaded a new INI that supports 1202 but I cant
get it to work. I swear I tried everything.
https://github.com/asmtron/rdpwrap/blob/master/binary-download.md
Autoupdate does not concider x86 because the author only updates x64
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/1526#issuecomment-922124752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALA3MLOAIB775DYJLHBAUT3UCPCKLANCNFSM5EIDVV7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

After removing the update, I ran the unistall.bat file and did the install procedure again.  I just checked if I can log in remotely and is working.  Also make sure to add the rdpwrap directory to the exception list of anit-virus program.

Yeah, no luck. :(
Ariel Fernández
Director
[AFER IT & Internet Solutions]
El vie., 17 de septiembre de 2021 20:26, rm-2021 @.***>
escribió:

After removing the update, I ran the unistall.bat file and did the install
procedure again. I just checked if I can log in remotely and is working.
Also make sure to add the rdpwrap directory to the exception list of
anit-virus program.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/1526#issuecomment-922132098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALA3MLMBJNMFBKRPFQTK6NLUCPFDHANCNFSM5EIDVV7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

If the local session is being kicked out when logging in remotely then sound like  single session per user.   Uncheck the box  in rdpconf.exe

The local session gets kicked out when anyone logs in, that box is
unchecked.
I did saw it working before the update.
El vie, 17 sept 2021 a las 21:22, rm-2021 @.***>)
escribió:

If the local session is being kicked out when logging in remotely then
sound like single session per user. Uncheck the box in rdpconf.exe
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/1526#issuecomment-922144925,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALA3MLNF5YQHR6625Q3IW4DUCPLTPANCNFSM5EIDVV7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

https://github.com/stascorp/rdpwrap/issues/1509#issuecomment-921345297

Copy from x86 section [10.0.19041.1081] to [10.0.19041.1202]. This is working on 32bit 10.0.19041.1202!!!
Yes, author ignore x86 in updates.

[10.0.19041.1202]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=89D81
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=0D212
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=19105
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=1E98C
SLInitFunc.x64=New_CSLQuery_Initialize
LocalOnlyPatch.x86=1
LocalOnlyOffset.x86=B66B9
LocalOnlyCode.x86=jmpshort
DefPolicyPatch.x86=1
DefPolicyOffset.x86=3E899
DefPolicyCode.x86=CDefPolicy_Query_eax_rcx
SLInitHook.x86=1
SLInitOffset.x86=68809
SLInitFunc.x86=New_CSLQuery_Initialize
[10.0.19041.1202-SLInit]
bInitialized.x64      =107108
bServerSku.x64        =10710C
lMaxUserSessions.x64  =107110
bAppServerAllowed.x64 =107118
bRemoteConnAllowed.x64=107120
bMultimonAllowed.x64  =107124
ulMaxDebugSessions.x64=107128
bFUSEnabled.x64       =10712C
bInitialized.x86      =D1A04
bServerSku.x86        =D1A08
lMaxUserSessions.x86  =D1A0C
bAppServerAllowed.x86 =D1A14
bRemoteConnAllowed.x86=D1A1C
bMultimonAllowed.x86  =D1A20
ulMaxDebugSessions.x86=D1A24
bFUSEnabled.x86       =D1A28

Copy from x86 section [10.0.19041.1081] to [10.0.19041.1202]. This is working on 32bit 10.0.19041.1202!!!
Yes, author ignore x86 in updates.

Thanks! That works!

Best to submit the 32 bit (x86) DLL and have it assessed properly; only the 64 bit (x64) DLL had been submitted.
But really, it's about time everyone used 64 bit these days; why would anybody install 32 bit Windows now, are there some kind of edge cases out there that won't work with 64 bit Windows?
As less and less people use 32 bit Windows, this is likely to be a problem going forward; that is, unless someone whom continues to use 32 bit Windows is happy to keep submitting their termsrv.dll when it is updated (and perhaps more than one person as there are variations of releases/builds as well.

[10.0.19041.1202]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=89D81
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=0D212
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=19105
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=1E98C
SLInitFunc.x64=New_CSLQuery_Initialize
LocalOnlyPatch.x86=1
LocalOnlyOffset.x86=B66B9
LocalOnlyCode.x86=jmpshort
DefPolicyPatch.x86=1
DefPolicyOffset.x86=3E899
DefPolicyCode.x86=CDefPolicy_Query_eax_rcx
SLInitHook.x86=1
SLInitOffset.x86=68809
SLInitFunc.x86=New_CSLQuery_Initialize
[10.0.19041.1202-SLInit]
bInitialized.x64 =107108
bServerSku.x64 =10710C
lMaxUserSessions.x64 =107110
bAppServerAllowed.x64 =107118
bRemoteConnAllowed.x64=107120
bMultimonAllowed.x64 =107124
ulMaxDebugSessions.x64=107128
bFUSEnabled.x64 =10712C
bInitialized.x86 =D1A04
bServerSku.x86 =D1A08
lMaxUserSessions.x86 =D1A0C
bAppServerAllowed.x86 =D1A14
bRemoteConnAllowed.x86=D1A1C
bMultimonAllowed.x86 =D1A20
ulMaxDebugSessions.x86=D1A24
bFUSEnabled.x86 =D1A28

btw that INI above is missing these three entries:
SingleUserPatch.x86
SingleUserOffset.x86
SingleUserCode.x86

I will publish a new ini tomorrow to incorporate the sections in full from https://github.com/stascorp/rdpwrap/issues/1501
(it will cover both x86 sections for each of 10.0.19041.1081 and 10.0.19041.1202 on the basis that the issue 1501 reports it as working and that both x64 versions were identical for these builds)

INI file updated as promised

Best to submit the 32 bit (x86) DLL and have it assessed properly; only the 64 bit (x64) DLL had been submitted.
But really, it's about time everyone used 64 bit these days; why would anybody install 32 bit Windows now, are there some kind of edge cases out there that won't work with 64 bit Windows?
As less and less people use 32 bit Windows, this is likely to be a problem going forward; that is, unless someone whom continues to use 32 bit Windows is happy to keep submitting their termsrv.dll when it is updated (and perhaps more than one person as there are variations of releases/builds as well.

Old PC with 2gb ram in organisation, updated to Windows 10 32 bit

Old PC with 2gb ram in organisation, updated to Windows 10 32 bit

Really, I know that there is a labor cost in setting up a new PC, but a newer PC would cost peanuts with more RAM and a better CPU (amongst other things).  I could understand if there was a hardware component that is legacy and wouldn't work in a newer PC.

Remove KB5005565, it helped me

My friends,
I could not wait anymore so I ended up adding 2 old Athlon 250 x2 CPU.
But I agree on removing that one KB, it also breaks shared printers.
Ariel Fernández
Director
[AFER IT & Internet Solutions]
El mié., 22 de septiembre de 2021 04:33, mymobilmail <
@.***> escribió:

Remove KB5005565, it helped me
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/1526#issuecomment-924664009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALA3MLOQSTVGXPCOV6Q2HCTUDGBEXANCNFSM5EIDVV7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


vim9script: different for-loop behavior at script level and inside a def function

Steps to reproduce
This is a continuation of the discussion in https://github.com/vim/vim/issues/11094#issuecomment-1274963516.
The following for-loop works differently at script level vs inside a def function:
vim9script

# Filter out all odd numbers in each sublist
var list: list<list<number>> = [[1], [1, 2], [1, 2, 3], [1, 2, 3, 4], [1, 2, 3, 4, 5]]
for i in list
    filter(i, (_, n: number): bool => n % 2 == 0)
endfor

# [[], [2], [2], [2, 4], [2, 4]]
echo list

Everything works as expected. However, inside a def function, the above snippet doesn't compile:
vim9script
def Test()
    var items: list<list<number>> = [[1], [1, 2], [1, 2, 3], [1, 2, 3, 4], [1, 2, 3, 4, 5]]
    for i in items
        filter(i, (_, n: number): bool => n % 2 == 0)
    endfor
    echo items
enddef
Test()

Result:
Error detected while compiling /tmp/test.vim[34]..function <SNR>22_Test:
line    3:
E1307: Argument 1: Trying to modify a const list<number>

What does this mean? Does Vim copy each item inside the list? That would be quite inefficient. The items could be anything.
The following works but looks more complicated:
def Test()
    var items: list<list<number>> = [[1], [1, 2], [1, 2, 3], [1, 2, 3, 4], [1, 2, 3, 4, 5]]
    for i in items
        var item: list<number> = i
        filter(item, (_, n: number): bool => n % 2 == 0)
    endfor
    echo items
enddef

Expected behaviour
I think there shouldn't be a difference whether the same lines of code are called inside a def function or at script level.
Version of Vim
9.0.723
Environment
Linux
Logs and stack traces
No response

The loop variable is considered a constant, you can't change it.
But if the value is a list, I suppose it is OK the change the contents of that list, as you intend to do here.
It's more like the loop variable is "final".
That means the compiled function does this wrong.


Use tantivy's ExistsQuery for fast fields

Description
Updates ExistsQuery in quickwit to use tantivy's ExistsQuery for the fast fields.
Fixes #3721
How was this PR tested?
Modified existing test to ensure they hit both use cases.

I openned it as a draft since it is pointing to an unmerged PR in tantivy at the moment. This should be updated before merging.

Ready for review.


Invalid syntax when executing script

When executing the script with or without arguments, using Python 2.7 or 3.6.3 on a Mac:
File "jsoncsv.py", line 51
elif type(value) is dict:
^
SyntaxError: invalid syntax

No errors at my end, similar environment as yours.
Have you modified the code? Can you post a code snippet and the command you are running?

I haven't modified anything, just copy past to json_to_csv.py.  Below are command examples.  I am running this on Mac in Bash, not sure if that's an issue.
$ python json_to_csv.py
File "json_to_csv.py", line 51
elif type(value) is dict:
^
SyntaxError: invalid syntax
$ python json_to_csv.py records response.json response.csv
File "json_to_csv.py", line 51
elif type(value) is dict:
^
SyntaxError: invalid syntax

FYI, as a test, I cloned your repository and the file runs fine.  I've compared the files between copy/paste of the code and the cloned file and they are identical.  Very odd, but at least I have a workaround.


YouTube Asking Users to Login to use Embedded Player in Plyr?!

All of the YouTube videos embedded with plyr on my website are showing this error. Has YouTube changed something on there end that is causing Plyr to get flagged as a bot?
My site is: https://jackklink.com. The first and second videos are YouTube embeds.

Well never mind. Now its not doing this anymore.

Ugh ok now it's doing it again. This seems to be occurring randomly regardless of whether or not I am signed into YouTube. I hope my site vistors aren't met with this error!


adding missing openapi comments to bankfeed endpoint.

adding missing openapi comments to bankfeed endpoint. fixes #102

Thanks @pjunior94 Great work 👍 Happy Hacktoberfest


update default SSL ciphers

suggested from https://wiki.mozilla.org/Security/Server_Side_TLS#Nginx_configuration_details and https://weakdh.org/sysadmin.html

Looks good, thanks!


Refactor polling controllers to remove legacy polling patterns and method usage

TODO: list affected controllers (TokenDetectionController, ...)

@MajorLift Would you mind updating the PR description to elaborate more on what you had in mind here?


Record.extends?

I am trying to use runtypes to validate the inputs to when drawing a bunch of shapes, their parameters are designed as follows:
const BaseShapeParams = Record({
    x: Number,
    y: Number,
    width: Number,
    height: Number,
    rotation: Number,
});

const RectParams = BaseShapeParams;
const EllipseParams = BaseShapeParams;

However, for a polygon type and a star type I would need two additional fields, sides and ratio. Right now I see no way of extending to an existing Record directly. And I would have to do this:
const PolygonParams = Record({
    x: Number,
    y: Number,
    width: Number,
    height: Number,
    rotation: Number,
    sides: Number,
});

const StarParams = Record({
    x: Number,
    y: Number,
    width: Number,
    height: Number,
    rotation: Number,
    sides: Number,
    ratio: Number,
});

Which produces a lot of redundant code. I understand that you could do something like this:
const baseShapeParams = {
    x: Number,
    y: Number,
    width: Number,
    height: Number,
    rotation: Number,
} as const;

const PolygonParams = Record({
    ...baseShapeParams,
    sides: Number,
});

But it would still be helpful to be able to extend on an existing Record type. What I imagine could've been done is something like:
// typescript's static analysis recognizes this successfully
interface InternalRecord<
  O extends { [_: string]: RuntypeBase },
  Part extends boolean,
  RO extends boolean
> extends Runtype<RecordStaticType<O, Part, RO>> {
      extends<T extends { [_: string]: RuntypeBase }>(fields: T): InternalRecord<O & T, Part, RO>;
}

const PolygonParams = BaseShapeParams.extends({
    sides: Number,
});
/**
 * InternalRecord<{
 *     x: Number;
 *     y: Number;
 *     width: Number;
 *     height: Number;
 *     rotation: Number;
 * } & {
 *     sides: Number;
 * }, false, false>
 */

Now available in v6.3.0 :wink:

What is the difference with the existing .And() method?

@KingDarBoja

.And() returns an Intersect, while .extend() returns a Record. This difference is also reflected in their Static types, i.e. the former is inferred like { ... } & { ... } and the latter is just { ... }.
.And() can return an invalid runtype such as Intersect(Record({ foo: Number }), Record({ foo: String }))  if fields are incompatible, while Record({ foo: Number }).extend({ foo: String }) detects it and notifies us as a compile-time error for each incompatible field, like Type 'string' is not assignable to type 'number'..
If the above error was @ts-ignored, it will result in a runtype which is just incompatible to the base runtype, not invalid, i.e. Record({ foo: String }).

Based on that explanation, seems like changing every .And() to .extend() provides more safety and better static types, which is something I have been looking for (never liked the inferred intersect when the type has several fields). Also enables better code organization if someone is looking to share runtypes between the server and the app using monorepo tools and such.
Outstanding job 🚀


Added code editor

fixes #635

@iamrahulmahato The checks failed are for Bubble-game and not my project.

I am closing this PR because all of the checks have failed. Please create a new PR
.


Added new Treesitter capture names

A recent Treesitter change left some tokens not matched by capture names. This is an attempt to add those new capture names.
I let the old ones stay since I imagined they would be useful to others not running the most recent Neovim version(s).

I did similar changes recently, and it seems your list completes mine. I'll share my diff if you want, so note that I did a full change, instead of just adding the new capture names. Don't hesitate to reach out if you have questions about it / want some help.
diff --git a/lua/gruvbox-baby/theme.lua b/lua/gruvbox-baby/theme.lua
index e405763..a103377 100644
--- a/lua/gruvbox-baby/theme.lua
+++ b/lua/gruvbox-baby/theme.lua
@@ -91,6 +91,7 @@ function M.setup(config)
     EndOfBuffer = { fg = c.background },

     -- Tree Sitter
+    ["@attribute"] = { fg = c.orange, style = "bold" },
     ["@boolean"] = { fg = c.orange },
     ["@define"] = { fg = c.red },
     ["@comment"] = { fg = c.comment, style = config.comment_style },
@@ -104,43 +105,44 @@ function M.setup(config)
     ["@string"] = { fg = c.soft_green, style = config.string_style },
     ["@character"] = { fg = c.soft_green },
     ["@number"] = { fg = c.magenta },
-    ["@namespace"] = { fg = c.light_blue },
+<EMAIL_ADDRESS>= { fg = c.magenta },
<EMAIL_ADDRESS>= { fg = c.soft_yellow },
     ["@function"] = { fg = c.soft_yellow, style = config.function_style },
<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = config.function_style },
<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = config.function_style },
<EMAIL_ADDRESS>= { fg = c.soft_yellow },
-    ["@parameter"] = { fg = c.blue_gray, style = "nocombine" },
-<EMAIL_ADDRESS>= { fg = c.dark_gray },
-    ["@method"] = { fg = c.soft_yellow, style = config.function_style },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = config.function_style },
-    ["@field"] = { fg = c.light_blue },
     ["@property"] = { fg = c.blue_gray },
     ["@constructor"] = { fg = c.clean_green, style = "nocombine" },
-    ["@conditional"] = { fg = c.red },
-    ["@repeat"] = { fg = c.red },
-    ["@label"] = { fg = c.blue_gray },
     ["@keyword"] = { fg = c.red, style = config.keyword_style },
-<EMAIL_ADDRESS>= { fg = c.red, style = config.keyword_style },
+<EMAIL_ADDRESS>= { fg = c.red },
+<EMAIL_ADDRESS>= { fg = c.red },
<EMAIL_ADDRESS>= { fg = c.red, style = config.keyword_style },
<EMAIL_ADDRESS>= { fg = c.red },
+<EMAIL_ADDRESS>= { fg = c.red },
+<EMAIL_ADDRESS>= { fg = c.red, style = config.keyword_style },
+    ["@label"] = { fg = c.blue_gray },
+    ["@markup"] = { fg = c.soft_yellow },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "bold" },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "italic" },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "underline" },
+<EMAIL_ADDRESS>= { fg = c.medium_gray, style = "strikethrough" },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow },
+    ["@module"] = { fg = c.light_blue },
     ["@operator"] = { fg = c.foreground },
-    ["@exception"] = { fg = c.red },
     ["@type"] = { fg = c.clean_green },
<EMAIL_ADDRESS>= { fg = c.blue_gray },
<EMAIL_ADDRESS>= { fg = c.orange },
<EMAIL_ADDRESS>= { fg = c.orange },
     ["@structure"] = { fg = c.blue_gray },
+    ["@uri"] = { fg = c.soft_yellow },
     ["@variable"] = { fg = c.light_blue, style = config.variable_style },
<EMAIL_ADDRESS>= { fg = c.blue_gray },
-    ["@text"] = { fg = c.soft_yellow },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "bold" },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "italic" },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = "underline" },
-<EMAIL_ADDRESS>= { fg = c.medium_gray, style = "strikethrough" },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow },
-<EMAIL_ADDRESS>= { fg = c.soft_yellow },
-    ["@uri"] = { fg = c.soft_yellow },
+<EMAIL_ADDRESS>= { fg = c.blue_gray, style = "nocombine" },
+<EMAIL_ADDRESS>= { fg = c.dark_gray },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = config.function_style },
+<EMAIL_ADDRESS>= { fg = c.soft_yellow, style = config.function_style },
+<EMAIL_ADDRESS>= { fg = c.light_blue },
     ["@tag"] = { fg = c.clean_green },
<EMAIL_ADDRESS>= { fg = c.comment },
<EMAIL_ADDRESS>= { fg = c.soft_yellow },

Thank you!

My knowledge of Treesitter terminology is not that deep. Where does your capture names, that doesn't exist in my PR, come from @gheritarish?
I did a few visual diffs for where the highlighting were different after the Treesitter update and were satisfied by my changes.

I went into the code of the commit you linked mostly @karloskar. Especially for stuff like python decorators, which are now under the @attribute capture according to the CONTRIBUTING.md file. That, and the code modifications for python file.
Mostly, after a nvim upgrade I was lost with the new colors, and tried to fix it by modifying stuff as I went, until it seemed fine 😅
But your update is great and enough, the only color change before / after is python decorator, going from orange to purple. Is that a problem? Probably not, just a question of habit.

There seem to be a specific capture for Python attributes already in place here. And my decorators are orange :sweat_smile:


Will zerossl-bot client be supported in the future without snap?

One of the reasons we are willing to subscribe to zerossl is cerbot-auto not being supported because they are moving to snap but I am reading issues and just installed zerossl-bot and get this message:
Your system is not supported by certbot-auto anymore.
certbot-auto and its Certbot installation will no longer receive updates.
You will not receive any bug fixes including those fixing server compatibility
or security problems.

"zerossl-bot" is NOT an ACME client by itself. It's just a very simple wrapper script with some ZeroSSL specific stuff, but it just runs Certbot. There are ways to install Certbot without snap, such as using pip. No need to switch to ZeroSSL.


Add Makefile for clean up tasks, etc

Current coverage is 98.86% (diff: 100%)

Merging #935 into master will not change coverage

@@             master       #935   diff @@
==========================================
  Files            26         26
  Lines          1413       1413
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1397       1397
  Misses           16         16
  Partials          0          0

Powered by Codecov. Last update 4c3d5f0...fc6eb2c


add --without-masquerade option to weave expose

Add the ability to skip creation of MASQUERADE rules via iptables when executing the weave expose command.

First PR to the weave project ... all comments/suggestions/etc... appreciated.

Updated to address comments.  Let me know if there is anything else.

@leprechau Thanks for your contribution! Mind explaining why someone would want to skip the creation of the NAT rules?

We’re using weave in a standard Docker environment without k8s and want external systems (not part of the weave network) to see the container IP and not the IP of the host.
In our specific case we are using a helper written in-house (https://github.com/myENA/aardvark) to publish container routes and update the default route within the container when it starts.

@leprechau Wouldn't --no-masq-local from #3298 be enough for your case?

@brb Trying now ... I see the flag in prog/weaver/main.go but didn't see it listed in the actual weave help usage.  If it works I'll send a much smaller PR adding the option to the usage docs.

So, the --no-masq-local does not seem to work for my particular case.
What I did ...

Reset weave via weave reset on all test nodes
Restarted weave

weave launch --password xxxxxx --no-detect-tls --ipalloc-range <IP_ADDRESS>/22 --no-dns --without-dns --proxy=false --no-masq-local

Created a new docker network using the weave driver

docker network create -d weave --subnet=<IP_ADDRESS>/22 --gateway <IP_ADDRESS> app

Started a test container on the app network

@brb Am I missing something in how the --no-masq-local option is supposed to work?  The default route in my container is the weave bridge but the traffic is still NAT'd through the host unless I completely remove the MASQ rules.

@leprechau

Result ... traffic leaving the container <IP_ADDRESS> still shows up outside of the weave network as being NAT'd through the host <IP_ADDRESS>

OK, I misread your requirement - I thought that you want external systems to access containers w/o NAT (this is what the --no-masql-local for). Unfortunately, "traffic leaving the container" (I believe = "traffic originating from the container") will be NAT'ed.

@brb yes, I'm wanting hosts outside of the weave network to see traffic that originates from a container on the weave network to see the weave assigned IP address of the container.

@bboreham @brb Awesome, thank you both as well!


Added smb protocol with test

This solves https://github.com/quantumblacklabs/kedro/issues/765
Description
Added else for filesystems that contain "netloc" but are not cloud protocols.
Development notes
Added a simple test to ensure the username, password, host and port are returned in the path of smb files
Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[x] Updated the documentation to reflect the code changes
[x] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[x] Added tests to cover my changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

It appears the unit tests are failing because they are timing out the docs_linkcheck is failing because of some url on documentation about AWS that no longer exists.
Should I modify the url related to AWS?
Please let me know if you have any questions. thanks!

It appears the unit tests are failing because they are timing out and the docs_linkcheck is failing because of some url on documentation about AWS that no longer exists.
Should I modify the url related to AWS?
We're fixing the broken link in a different PR already so don't worry about it - when it's merged to master you can update this PR and it should fix it. And the unit tests timing out is probably just a random error, so re-rerunning the job (which will happen if you do another commit) will probably fix it. So probably nothing to worry about here 🙂

This looks great to me, thank you for your contribution @lucasjamar! I'm not very familiar with these protocols so will try and get some people who are more familiar to take a look.
Please could you explain to a dummy why the whole username:password@myhost:8080 get passed as a path here when it doesn't for s3 etc.?

To follow-up on your question @AntonyMilneQB , id be interested to know why the host and port are ignored for hdfs if someone knows the answer. I would have thought these are important pieces of information.

@lucasjamar @AntonyMilneQB @lorenabalan I think generally our strategy when parsing filepath and instantiate the underlying fsspec.filesystem is:

Parse path and protocol from filepath attribute of the dataset. The reason we have our own _parse_path function and not use the underlying fsspec.utils.infer_storage_options was due to how infer_storage_options was treating abl/adfs differently from s3/gcs: https://github.com/dask/adlfs/issues/45. So to ensure parsing consistency, we roll our own _parse_path. Since this deals with path and protocol only, everything else is discarded.
host & port are added to fs_args as @lucasjamar has noticed.
username & password should be added to credentials.

When we instantiate fsspec.filesystem, e.g. in a dataset, we will pass everything as kwargs to fsspec, so the net effect would be similar to passing a string containing everything to fsspec and allow fsspec to parse it with infer_storage_options.
What I'd suggest is maybe looking into why you need to specify a dummy url and fix that instead so you can write:
filepath: smb://my/file/path/
fs_args:
    host:
    port:
credentials: my_smb_dataset_credentials

Thank you very much for this @limdauto! Super helpful 👍  Just for completeness, when I was looking to this earlier I also found the original commit where we moved from fsspec's infer_storage_options to our own implementation. Looking through infer_storage_options, it is indeed very similar to our _parse_path.

@lucasjamar @AntonyMilneQB @lorenabalan I think generally our strategy when parsing filepath and instantiate the underlying fsspec.filesystem is:
* Parse `path` and `protocol` from `filepath` attribute of the dataset. The reason we have our own `_parse_path` function and not use the underlying `fsspec.utils.infer_storage_options` was due to how `infer_storage_options` was treating `abl/adfs` differently from `s3/gcs`: [dask/adlfs#45](https://github.com/dask/adlfs/issues/45). So to ensure parsing consistency, we roll our own `_parse_path`. Since this deals with `path` and `protocol` only, everything else is discarded.

* `host` & `port` are added to `fs_args` as @lucasjamar has noticed.

* `username` & `password` should be added to `credentials`.

When we instantiate fsspec.filesystem, e.g. in a dataset, we will pass everything as kwargs to fsspec, so the net effect would be similar to passing a string containing everything to fsspec and allow fsspec to parse it with infer_storage_options.
What I'd suggest is maybe looking into why you need to specify a dummy url and fix that instead so you can write:
filepath: smb://my/file/path/
fs_args:
    host:
    port:
credentials: my_smb_dataset_credentials

@limdauto Thanks a lot for the helpful answer! As mentioned in an earlier comment, urlsplit interprets the first part of the path after :// as the network location even though it belongs to the path. For this reason, this information gets lost. Unfortunately, I dont see any alternative than the dummy url I mentioned or the implementation proposed. Or am I getting this wrong and there is a better solution? Thanks :)

Amazing, thanks very much @limdauto.
@lucasjamar I think this means we don't need this PR any more?


Use ayu-colors 4.0.0 (improved contrast)

Fixes #23
Updated screenshots:

Does the screenshot of dark theme looks good enough on your display now @batjko?

@pahen It's subtle, but definitely makes a difference, so I'd certainly be happy with this.
Nice work. 👍

Let me know if you want to me to help you out as a maintainer of this theme if you're busy @teabyii
It would be awesome if we can move the repo https://github.com/ayu-theme to make it "official" if @dempfi and @k15a are interested in that.

I has invited you to be the collaborator of this repo. Thank you for helping.

Thanks @teabyii :)
Can you give me permissions to publish to Extension Marketplace too?
I think the way to do it is explained here: https://github.com/Microsoft/vscode/issues/23801#issuecomment-292512395. The email you should add there as a contributor is<EMAIL_ADDRESS>

@pahen I've added you to the contributor in vscode marketplace. You can try to publish a new version.

Thanks! It worked :) Published 0.5.0 now 🎉


Upgrading Jackson Dependency

This version of Jackson causes serialization error w/ ObjectNode.

Looks like this didn't build out for java 7? The error doesn't look related tho should we try a rebuild?


unbox snippets when rendering in tip

Hints only contain a single code snippets (they could have more but they shouldn't). We waste space by wrapping the snippet in a segment that adds padding. Don't wrap snippets when rendering tutorial hint.
before:

after:

Could probably be done in css...


Dockerize age-plugin-yubikey and rage

Dependencies to system deps means tools might break when those libs get updated.
Dockerizing the tool provides a snapshot which will still work.

Hi @str4d, can we move forward with this PR or would you rather close it?

I don't think it makes sense for the Dockerfile in this repository to build rage; there are legitimate use cases for using age-plugin-yubikey with other age implementations (such as age itself). But if the rage build is removed, then I'm not sure what purpose the Dockerfile serves, as AFAICT it would not result in a complete / standalone age + age-plugin-yubikey environment. Unless there is an alternate approach that enables Dockerfiles across multiple repositories to be composed (I am not much of a Docker user so I'm not familiar with how Dockerfiles interact), I'm inclined to close this and leave environment creation to downstream packagers.


Responsive 3-page Portfolio Website Template

Project title
A clean and responsive 3-page Portfolio Website Template
Project Description
A clean and responsive 3-page Portfolio Website Template
Stack:

[x] Html
[x] CSS

@gurjeetsinghvirdee Please assign


refactor(salt): rewrite salt installer script

This PR introduces a newer `salt/installer.sh bash script which replaces previous scripts:-
Obsoluted:

install.sh :
ea/package-mgr.sh:

This PR has no impact on salt artifacts which install OpenSDS - only bash/directory impacts.
The OpenSDS installation using salt has been verified on Ubuntu 18.04 successfully.

No conflicts.

Rebased with master


fix(radio-button): update styles to audit spec

Closes #1926
Changelog
Changed

focus outline

Removed

radio button group margin.

@aledavila to be consistent can we say "Radio button label" instead of the word "heading?" We don't use that anywhere else.

Not sure if it's just me, but I don't see a focus state in Safari

Also, we're not offering this to anybody right? It's whack.

One last point, and this applies to most of the components... can we vertically center them in the component display window? So many of them ride high.


Uncaught Error: No screen line exists when converting buffer row to screen row

I'm oftentimes getting an error and the stack trace is as follows:
Uncaught Error: No screen line exists when converting buffer row to screen row /Applications/Atom.app/Contents/Resources/app/src/display-buffer.js:998
module.exports.DisplayBuffer.screenPositionForBufferPosition /Applications/Atom.app/Contents/Resources/app/src/display-buffer.js:998
module.exports.Marker.getHeadScreenPosition /Applications/Atom.app/Contents/Resources/app/src/marker.js:211
Marker /Applications/Atom.app/Contents/Resources/app/src/marker.js:45
module.exports.DisplayBuffer.getMarker /Applications/Atom.app/Contents/Resources/app/src/display-buffer.js:1190
module.exports.DisplayBuffer.markBufferRange /Applications/Atom.app/Contents/Resources/app/src/display-buffer.js:1222
module.exports.TextEditor.markBufferRange /Applications/Atom.app/Contents/Resources/app/src/text-editor.js:1198
LinterView.display linter-view.coffee:180
LinterView.processMessage linter-view.coffee:159
(anonymous function) linter-view.coffee:1
(anonymous function) linter-view.coffee:142
Linter.processMessage linter.coffee:151
(anonymous function) linter.coffee:127
BufferedProcess.triggerExitCallback /Applications/Atom.app/Contents/Resources/app/src/buffered-process.js:52
(anonymous function) /Applications/Atom.app/Contents/Resources/app/src/buffered-process.js:59
(anonymous function) /Applications/Atom.app/Contents/Resources/app/src/buffered-process.js:105
emit events.js:129(anonymous function) net.js:461

It's really obnoxious because when it gets into this state it keeps throwing the error (seems to happen whenever I hit Command+S to save) until I exit and reopen Atom.

There hasn't been any more reports of this. Are you still experiencing it?

I haven't seen it in a while, closing for now; I'll re-open if it crops up again

I'm getting it again.
Not really sure how to repro it, but I've got a scss file open in one pane and js in 2 other panes.
Atom Version: 0.201.0
System: Mac OS X 10.10.3
Thrown From: linter package, v0.12.6
Stack Trace
Uncaught Error: No screen line exists when converting buffer row to screen row
At /Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1047

Error: No screen line exists when converting buffer row to screen row
  at DisplayBuffer.module.exports.DisplayBuffer.screenPositionForBufferPosition (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1047:17)
  at Marker.module.exports.Marker.getHeadScreenPosition (/Applications/Atom.app/Contents/Resources/app.asar/src/marker.js:174:33)
  at new Marker (/Applications/Atom.app/Contents/Resources/app.asar/src/marker.js:34:41)
  at DisplayBuffer.module.exports.DisplayBuffer.getMarker (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1265:20)
  at DisplayBuffer.module.exports.DisplayBuffer.handleBufferMarkerCreated (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1592:25)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:5:61
  at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:82:11)
  at TextBuffer.module.exports.TextBuffer.markerCreated (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:1253:20)
  at MarkerStore.module.exports.MarkerStore.createMarker (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/marker-store.js:313:21)
  at MarkerStore.module.exports.MarkerStore.markRange (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/marker-store.js:148:19)
  at TextBuffer.module.exports.TextBuffer.markRange (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:604:31)
  at DisplayBuffer.module.exports.DisplayBuffer.markBufferRange (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1297:41)
  at TextEditor.module.exports.TextEditor.markBufferRange (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1245:59)
  at LinterView.createMarker (/Users/braden/.atom/packages/linter/lib/linter-view.coffee:190:22)
  at LinterView.display (/Users/braden/.atom/packages/linter/lib/linter-view.coffee:227:17)
  at LinterView.processMessage (/Users/braden/.atom/packages/linter/lib/linter-view.coffee:178:8)
  at LinterView.processMessage (/Users/braden/.atom/packages/linter/lib/linter-view.coffee:1:1)
  at /Users/braden/.atom/packages/linter/lib/linter-view.coffee:164:14
  at LinterScssLint.Linter.processMessage (/Users/braden/.atom/packages/linter/lib/linter.coffee:241:5)
  at /Users/braden/.atom/packages/linter/lib/linter.coffee:173:8
  at triggerExitCallback (/Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:213:47)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:220:18
  at Socket.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:98:18)
  at emitOne (events.js:82:20)
  at Socket.emit (events.js:166:7)
  at Pipe.close (net.js:464:12)

Commands
     -1:27.7.0 atomic-emacs:previous-line (atom-text-editor.editor.kill-ring.is-focused)
  3x -1:27.5.0 atomic-emacs:next-line (atom-text-editor.editor.kill-ring.is-focused)
     -1:26.8.0 editor:move-to-end-of-line (atom-text-editor.editor.kill-ring.is-focused)
  2x -1:26.5.0 editor:newline (atom-text-editor.editor.kill-ring.is-focused)
  9x -1:21.9.0 core:backspace (atom-text-editor.editor.kill-ring.is-focused.autocomplete-active)
     -1:15.8.0 editor:newline (atom-text-editor.editor.kill-ring.is-focused)
     -1:14.1.0 atomic-emacs:next-line (atom-text-editor.editor.kill-ring.is-focused)
     -1:09.0 core:close (atom-text-editor.editor.kill-ring.is-focused)
     -1:08.2.0 fuzzy-finder:toggle-file-finder (atom-text-editor.editor.kill-ring.is-focused)
     -1:06.5.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -1:06.3.0 core:confirm (atom-text-editor.editor.mini.is-focused)
  6x -1:05.7.0 atomic-emacs:next-line (atom-text-editor.editor.kill-ring.is-focused)
     -0:27.6.0 settings-view:open (atom-text-editor.editor.kill-ring.is-focused)
  2x -0:20.4.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -0:19.9.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:02.3.0 core:close (div.settings-view.pane-item)

Config
{
  "core": {
    "ignoredNames": [
      ".git",
      ".svn",
      ".DS_Store",
      "build"
    ],
    "themes": [
      "quantum-ui",
      "base16-tomorrow-dark-theme"
    ],
    "useReactEditor": true,
    "disabledPackages": [
      "editor-stats",
      "toggle-tabs",
      "term2",
      "sublime-tabs",
      "tabs",
      "emacs-mode",
      "terminal-status",
      "linter-jshint"
    ]
  },
  "linter": {
    "subtleLinterErrors": [
      "rubocop"
    ],
    "lintOnChangeInterval": 500,
    "ignoredLinterErrors": [
      "jshint"
    ]
  }
}

Installed Packages
# User
atom-beautify, v0.27.6
atom-lint, v0.20.1
atomic-emacs, v0.5.1
custom-title, v0.7.1
emacs-mode, v0.0.29
language-haml, v0.20.0
linter, v0.12.6
linter-coffeelint, v0.2.4
linter-haml, v0.4.0
linter-rubocop, v0.2.7
linter-scss-lint, v1.0.4
linter-tidy, v1.0.1
pane-info, v1.0.0
pretty-json, v0.3.3
quantum-ui, v0.2.2
react, v0.11.8
tag, v0.3.0

# Dev
No dev packages


/vanish command

I can try to figure out which packets might need to be adjusted to implement this.

Would vanish functionality need to be implemented at the proxy layer to some extent? I don't have full insight into what happens across all layers and thus I don't know if that would be necessary.

Tab List Management: Tab list packets might need to be altered to reflect vanished players only to those with the necessary permissions.
Packet Filtering: Various packets from vanished clients would need to be filtered and only sent to players with permission to see them.
State Updates: Additional packets would be required upon vanish/unvanish to update all clients with the player's new state. This ensures players don't appear to "float in the air" when vanishing, for example.
Collision and Interaction:

Auto-completion and tab suggestions might need to exclude vanished players.
Entity collision, arrow collision (even though not implemented yet), and player PvP would need to respect the vanished state.

Would vanish functionality need to be implemented at the proxy layer to some extent?

We might need a version of the BroadcastGlobal server to proxy packet that can exclude multiple users (the ones with permissions to view vanished players) when broadcasting the packets for removing the player and another version which only includes admins when broadcasting the movement packets of vanished players. Other than that, I don't think there would need to be special handling of vanished players at the proxy

Would vanish functionality need to be implemented at the proxy layer to some extent?

We might need a version of the BroadcastGlobal server to proxy packet that can exclude multiple users (the ones with permissions to view vanished players) when broadcasting the packets for removing the player and another version which only includes admins when broadcasting the movement packets of vanished players. Other than that, I don't think there would need to be special handling of vanished players at the proxy

yeaaaa ok this is getting complicated fast 🤔

I think the group solution makes more sense it would allow us to do something like the Audiences system in KyoriAdventureAPI but with packets.
So in an event, you could have the "player" audience which is the basic one, a "spectator" or "camera" audience which could, for instance, see players highlighted, see different scoreboards with stats about the current state of the event, or see particle trails indicating where arrows came from etc...

Well, I think having a vanish command is important. I don't think that it should be prioritized because there are other much more important things to do, so I'm going to close this issue for now. But when other things are completed, I am down to reopen this.


add default attribute to set the 'user' attribute for userdefaults

tl;dr - similar to what #21 wants, but less specific - allow the "user" attribute to be set/overridden globally.
trying to use mac_os_x_userdefaults with kitchenplan resulted in...interesting failures.    (kitchenplan's own defaults write doesnt support notifies sanely - and I didnt want to rewrite it).    turns out kitchenplan runs solo under sudo.   so my user level settings were getting set for root, instead of me.    kitchenplan does set a correct node['current_user'], though, so I don't need a mixin to get the right user.
To avoid any change to default behaviors, updated the userdefaults :user attribute to consume the new cookbook attribute.  This still defaults to nil, so should have zero impact.
For users who want to make the change - simply override node['mac_os_x']['user'] in your role/attributes/cookbook or whatever you prefer.   You can even use the cool chef 12 homebrew mixin - http://jtimberman.housepub.org/blog/2014/12/29/chef-12-homebrew-user-mixin/  :)

This is a useful feature. Can this be merged please?

@jtimberman  thoughts?

This may cause a no method found exception on nil because chef loads attributes files after it loads resource files.
It's probably safer and more sane to use the homebrew user mixin instead for the default.

good point - I'll see if I can find time to figure out a better route to make this happen.  (though "works for me!" haha)
I'll have to look at the homebrew user mixin again - I probably went this route out of simplicity though.


make shared_partition work for tenants

+1 - This patch works for me.  I set the v_method to LSI and with this patch I am able to specify the partition that I want the configuration created in - other than shared.


Hyperlinking to DOI?

URLs are useful, but so are DOIs. Many of my bibtex entries have a DOI but not a URL, since you can just make a URL by putting the DOI after http://dx.doi.org/  -- JabRef developers discussed it here.
I'm trying to do this, but there's no BIBTEXDOI variable. I don't want to create URLs in my database if I don't have to. Actually, it might make more sense to have a

the way it works with class="url" already, but which concatenates http://dx.doi.org/ and the doi field. into the href of the link.

No need to create urls in your database. For printing  just combine class="doi" and use bibtexVar, which is a way to access variables within the attributes of html elements. Below is an example of creating urls based on your doi field.

    https://dx.doi.org/

Let me know if this works for you.

That code doesn't work (crashes with an Uncaught TypeError: Cannot read property 'toUpperCase' of undefined) in a template (that's how I'm doing the formatting).
It's hard for me to debug it (give you the precise line), since the JavaScript is inline. I have made some hard-coded changes to make it work on the new Google Sites, which also sandboxes inline JavaScript, making debugging virtually impossible.
In the code that handles bibtexVar, it doesn't seem like DOI is one of the keys it looks for?

When did you last update your bibtex_js? Commit: c275c8517eef563696ac7bf0e6283cee51d5b6f8 had a bug fix when using .toUpperCase().
I made it work on the new Google Sites using the embedded feature and displaying the doi here. I didn't have to make any changes for it to work.
If you have a bibtex entry with "doi={value}" then "doi" will be one of the keys. The keys are generated from the bibtex entry.

Thanks! I updated and it is working nicely!
When I went to apply CSS to the H1 headers generated by the Academic Style (Google Sites embed doesn't inherit the site's CSS unfortunately, or at least I did figure out how to do it...), I noticed they have a strange class (undefined):

Refereed Journal Articles

It's not a big deal, since my CSS was for all H1's, but it looks like a bug.

I am struggling to understand why I get this error:
jQuery.Deferred exception: $(...).attr.extra is undefined BibtexDisplay/this.createTemplate/<@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:416:23
each@http://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js:2:2713
each@http://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js:2:1001
BibtexDisplay/this.createTemplate@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:415:9
BibtexDisplay/this.createStructure@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:659:31
BibtexDisplay/this.createStructure@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:651:24
BibtexDisplay/this.createStructure@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:621:38
BibtexDisplay/this.displayBibtex@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:686:13
bibtex_js_draw/<@https://cdn.rawgit.com/pcooksey/bibtex-js/abeec72eb0501d2f6ae41ed4f778b4a60e75984f/src/bibtex_js.js:729:18
j@http://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js:2:29997
g/</k<@http://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js:2:30313
 undefined

I am sure to be missing something...

Amazing! Thanks.
Which is the suggested way to always load the latest version in master?

No problem.
Github no longer lets you access the raw files so you can't reference the latest version in master (stack overflow for discussion on this topic). The file you are using is actually being hosted through rawgit (nothing to do with github). They provide two options for links to files. One using the commit/tag (what you are using) and a relative path one. However, the relative link is suppose to be reserved for development and if you have too much traffic to your website they will blacklist you and/or throttle your connect to the file.
So, there is currently no option to doing this on the fly like there used to be.

What about using github pages?
If you enable it on master then you can access the files at pcooksey.gitbub.io/bibtex-js/src/bibtex_js.js if I am not wrong

I don't think its a good idea for me to host the most up-to-date master. I am not guaranteeing backwards compatibility as I may update or improve features in the future. I would not want to break someones website that was previously working correctly. In this way, they can continue using the old version without worrying about changes I make. (For example, my recent changes to the academic style breaks backwards compatibility).
As for the cdn, you can definitely use it, but the rawgit website clearly states not to use branches:

Use a specific tag or commit hash in the URL (not a branch). Files are cached permanently based on the URL. Query strings are ignored.

so that link may be throttled if too much traffic comes to your website.


Avoid pidfile creation race

This patch (on top of #17) avoids a race on pidfile creation by creating the pidfile before the JSON is processed.
Prior to this patch, the pidfile was created only after the JSON had been processed. This creates a race if the initiating process assumes the daemon is running if it the call to start the daemon has returned.
For instance, before the patch, this was happening around 50% of the time under OS-X:

Shell process calls go program
Go program forks
Parent writes JSON
Child consumes JSON
Parent quits
Shell process does something that checks for the existence of the PID file
Shell process determines daemon is not running, and starts course of action in result
Child writes PID file
Daemon starts running

This patch corrects the problem by not consuming the JSON (and thus preventing the parent from quitting) until the lock file has been created. As the parent now cannot quit until the child has consumed the JSON (or closed the FD by crashing), we know that the child will have created the PID file by the time the parent exits, so it safe for the called (step 6 in the above) to check for the PID file and determine the daemon is running (or at least was started).

Great work. Thank you very much!


Polynomial notation gets stuck formatting (e^9050179352)4459417553120140

When formatting (e^9050179352)4459417553120140, one of the Polynomial notation presets get stuck in this while loop:
https://github.com/MathCookie17/Eternal-Notations/blob/90013468b97b03c85ff19e63e67590b8d4f314d8/dist/eternal_notations.js#L10567-L10572
Stack trace:
e (https://mathcookie17.github.io/Eternal-Notations/break_eternity%20stuff/break_eternity.min.js#1)
value (https://mathcookie17.github.io/Eternal-Notations/break_eternity%20stuff/break_eternity.min.js#1)
value (https://mathcookie17.github.io/Eternal-Notations/break_eternity%20stuff/break_eternity.min.js#1)
value (https://mathcookie17.github.io/Eternal-Notations/break_eternity%20stuff/break_eternity.min.js#1)
formatDecimal (https://mathcookie17.github.io/Eternal-Notations/dist/eternal_notations.js#10568)
format (https://mathcookie17.github.io/Eternal-Notations/dist/eternal_notations.js#10530)
formatOne (https://mathcookie17.github.io/Eternal-Notations/webpage.js#76)
formatAll (https://mathcookie17.github.io/Eternal-Notations/webpage.js#47)
(global) (source-actor-server0.conn0.windowGlobal92341796865/source148#1)

Found whilst fuzzing notations with this command:
for (let x = 0; x < 100; x++) formatAll(Decimal.tetrate(10, 2 ** (64 * Math.random())))

It's the x = 3 Polynomial causing this issue. I have a hunch on how to fix it, but given how many errors you've found, I'm trying to make a testing script so I can do more thorough, automatic testing of thousands of random Decimals on random presets. How have you been doing your testing? Clearly you're doing something randomized, but I'm not sure what.

For random Decimals up to layer 2^64, which includes all Decimals through layer 9e15 and a little bit past, I've been running the following lines in a browser's console.
for (let x = 0; x < 100; x++) formatAll(Decimal.tetrate(10, 2 ** (64 * Math.random())))

for (let x = 0; x < 100; x++) formatAll(Decimal.randomDecimalForTesting(2 ** (64 * Math.random())))

for (let x = 0; x < 100; x++) formatAll(Decimal.randomDecimalForTesting(Number.MAX_SAFE_INTEGER))

for (let x = 0; x < 100; x++) formatAll(Decimal.fromComponents(Math.random() > 0.5 ? -1 : 1, Math.floor(10 ** (Math.random() * 16)), 10 ** (Math.random() * 16)))

For random Decimals up to the limit of break_eternity.js, I've been running the following instead:
for (let x = 0; x < 100; x++) formatAll(Decimal.tetrate(10, 2 ** (1023.999999 * Math.random())))

for (let x = 0; x < 100; x++) formatAll(Decimal.randomDecimalForTesting(2 ** (1023.999999 * Math.random())))

for (let x = 0; x < 100; x++) formatAll(Decimal.randomDecimalForTesting(Number.MAX_VALUE))

for (let x = 0; x < 100; x++) formatAll(Decimal.fromComponents(Math.random() > 0.5 ? -1 : 1, Math.floor(2 ** (Math.random() * 1023.999999)), 10 ** (Math.random() * 16)))

In all cases, replace 100 with the number of Decimals you want to test (per line).


On button pressed error after several minutes of uptime

I am using an arduino Mega 2560 as a controller for a radio stack I am building. I am using two rotary switches set up as buttons in mobiflight to change a mobiflight variable (though this was also an issue when I was using FUIPC free offsets). After a few minutes of running, when I try to use these switches, two or three of the positions will throw the error mobiFlightCache_OnButtonPressed: tuple.Item2.DataBoundItem is NULL
and the issue cant be resolved without stopping mobiflight and running it again, but then the issue starts again in a few minutes.
The affected pins that the buttons are plugged into are 22-27 if that may be relavent.
Details

MobiFlight Version: 9.0.0
FlightSim: MSFS2020

thanks for reporting - do the switches stop to work or do you only see the messages? are you editing any settings while you are in "Run" mode?

I will close this due to

inactivity and also because
we made a lot of updates, so this might be no issue anymore.
there were no other general complaints by the community, so this might be an individual config problem too.

Feel free to reopen and provide instructions on how to reproduce.


Manage custom DHCPD configuration directly by web interface.

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
All started after reading this issues https://github.com/opnsense/core/issues/5299 but probably exists more other examples.
We have many instance of virtualized OPNsense in our Datacenter and the most important feature that this wonderful Firewall miss is the ability to release, via its DHCPv4 Server, IPs addresses that are not in the subnet of the interface where OPNsense receives the DHCPv4 requests. For instance if we have 2 or more branch offices in which are present only cisco routers (one per office) and one of the host in that office make a DHCPv4 request, the cisco router forwards this request via dhcp relay to our OPNsense. The problem is that OPNsense is not able to process this request because it receives this request on an interface that is not directly connected (layer 2) to the host that needs to receive a specific ip address.
After some investigation I realized that isc-dhcp, used by OpnSense, is able to manage this kind of configuration but OpnSense web interface no.
Describe the solution you like
I made some modification to OPNSense source code to include some customized row to dhcdp.conf file used by isc-dhcp.
I created a new simple interface on OPNSense to add the "custom row" to an external file "dhcpd.conf.custom" and I applied to /usr/local/etc/inc/plugins.inc.d/dhcpd.inc some rows of code to add an include to dhcpd.conf that point to new file "dhcpd.conf.custom".
I think that this update will be usefull for all parameters that are presents on isc-dhcp but is not actually managed by OPNSense interface.
Additional context
On the attached file dhcp_custom.zip you will find:

/usr/local/www/services_dhcp_custom.php that contain the web interface
/usr/local/opnsense/mvc/app/models/OPNsense/Core/Menu/Menu.xml with the call to services_dhcp_custom.php
/usr/local/opnsense/mvc/app/models/OPNsense/Core/ACL/ACL.xml with the call to services_dhcp_custom.php
/usr/local/etc/inc/plugins.inc.d/dhcpd.inc an altered version to manage an automatic include of new dhcpd.conf.custom file to dhcp.conf file

Considering that nested ArrayFields aren't possible at the moment using MVC, maybe the existing /services_dhcp.php?if=lan&act=newpool could be changed to support arbitrary pool definitions? It looks like the current functionality allows for additional pools only within the interface subnet, and it pre-defines subnet and subnet mask which would have to be editable to support arbitrary subnet definition not related to the interface.
If ArrayField could be nested, this would be a good opportunity for it.
Since the config can support multiple subnets, and items within the subnets can be multiple too.
# A slightly different configuration for an internal subnet.
subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
  range <IP_ADDRESS> <IP_ADDRESS>;
  option domain-name-servers ns1.internal.example.org;
  option domain-name "internal.example.org";
  option routers <IP_ADDRESS>;
  option broadcast-address <IP_ADDRESS>;
  default-lease-time 600;
  max-lease-time 7200;
}

The model could looks like this:
<pools>
    <entries type="ArrayField">
        <subnet type="NetworkField">
            <required>Y</required>
        </subnet>
        <netmask type="NetworkField">
            <required>Y</required>
        </netmask>
        <keys>
            <entries type="ArrayField">
                <key_name type="OptionField">
                    <required>Y</required>
                    <optionvalues>
                        <option>range</option>
                        <option>option</option>
                        <option>default-least-time</option>
                        ...
                    </optionvalues>
                </key_name>
                <option_name type="TextField">
                    <required>N</required>
                </option_name>
                <key_value type="TextField">
                    <required>N</required>
                </key_value>
            </entries>
        </keys>
    </entries>
</pools>

Intending to allow definition of multiple subnet {} definitions, each with their own options (keys, values). They could be accessed on a separate page built with the MVC and utilize bootgrids to create and edit the pools.


perf(theme): ⚡ change in typography theming

BREAKING CHANGE:
now all keys except 'breakPoints' will be read as css properties

:tada: This PR is included in version 2.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


x/tools/gopls: updating dependency from CodeLens tries to run "go mod get"

Please answer these questions before submitting your issue. Thanks!
What did you do?
Clicked on an "upgrade dependency to ..." code lens in my go.mod
What did you expect to see?
The dependency upgaded.
What did you see instead?
Nothing happens, and in the logs I see:
[Error - 1:31:47 PM] Request workspace/executeCommand failed.
  Message: err: exit status 2: stderr: go mod get: unknown command
Run 'go help mod' for usage.

  Code: 0

Which seems to imply it's trying to run go mod get as opposed to go get.
Build info
golang.org/x/tools/gopls master
<EMAIL_ADDRESS>h1:buT1PyVEdCfPp6DtNTRsMCJiAWgXkxksEtNXgK3fo7c=
<EMAIL_ADDRESS>h1:WXkYYl6Yr3qBf1K79EBnL4mak0OimBfB0XUf9Vl28OQ=
<EMAIL_ADDRESS>h1:we8PVUC3FE2uYfodKH/nBHMSetSfHDR6scGdBi+erh0=
<EMAIL_ADDRESS>h1:KU7oHjnv3XNWfa5COkzUifxZmxp1TyI7ImMXqFxLwvQ=
<EMAIL_ADDRESS>h1:vcxGaoTs7kV8m5Np9uUNQin4BrLOthgV7252N8V+FwY=
<EMAIL_ADDRESS>h1:5Y8c5HBW6hBYnGEE3AbJPV0R8RsQmg1/eaJrpvasns0=
<EMAIL_ADDRESS>h1:E7g+9GITq07hpfrRu66IVDexMakfv52eLZ2CXBWiKr4=
<EMAIL_ADDRESS>h1:UoveltGrhghAA7ePc+e+QYDHXrBps2PqFZiHkGR/xK8=
<EMAIL_ADDRESS>h1:NSZPykBXJFCetGZykLAxaL6SIpvbVy/UFEniIfHAa8A=

Go info
go version go1.14.4 linux/amd64

Thanks for the report! I bet this was a mistake in a recent CL of mine (https://golang.org/cl/235619). Very surprised that no tests caught it, so I'll send a fix and a test.


several small updates

The diff is pretty to easy to read and mostly self explanatory.

That was my take away as well 😆


Update caching.md

Adding additional details on the need to use sessionStorage to ensure compatibility with 'Browser CA' in AAD CA.

@saudmish unfortunately we won't be able to accept this, but I've made a note update either this doc or the FAQ to clarify the relationship between MSAL cache and Azure AD session. Thanks again for bringing this up. Closing.


Change execv to Popen to avoid cmd backgrounding issue in windows.

This is the issue specifically: https://bugs.python.org/issue19124
This is also mentioned by Brett in #5

Cool, thanks for the fix. It will help to find the next Windows bug :-)


Investigate castastrophic forgetting

Hard to know why sometimes it works sometimes it does not. Nothing in the logs.
I am running more experients with less parameters variation and more repeats.

The notebook should be put in the saved_experiments folder. Also the data should be pushed to the repo.

The notebook should be put in the saved_experiments folder. Also the data should be pushed to the repo.

What do you mean by the data should be pushed to the repo ? Not the dataset ?

@RomainGoussault are @arthurPignet comments relevant from your perspective? Can you indicate if you plan to keep making updates to this PR or if we should merge it now?

The notebook should be put in the saved_experiments folder. Also the data should be pushed to the repo.

What do you mean by the data should be pushed to the repo ? Not the dataset ?

I meant, the results files should be pushed to the repos. Actually I did pushed the results.csv but I forget to push the individual results. (When we resolved this https://github.com/SubstraFoundation/distributed-learning-contributivity/issues/174, life will be easier :))

@arthurPignet you are right on the local_validation_score, it is global_actually. (we should rename the array score_matrix_per_partner, it's a bit confusing).
And as you said, it will be nice to have the local validation score, but I think we don't have it.

@bowni yes I still plan to improve the notebooks, sorry for the delay !.
And Thanks @arthurPignet for the very relevant comments !
The next steps are:

pushing all the results (all the folders to get the loss data)
see if we can get the local validation score but I think we can not.


ERD Diagrams not displaying after update

Description
I have created many ERD diagramas in a previous version of DBeaver Community edition (from 22.3.1 to 23.0.0) and when a try to open ERD diagrams then just don't show the tables.
I've found that editing the erd file in notepad changing the line (from erd builded in v22.3.1 )
<data-source id="postgres-jdbc-18587b23bf3-1fd32db3bf3e1b4c">

for
<data-source id="postgres-jdbc-186bc7c206a-426452715cf88d0">

and then refresh the ERD with F5, they appear newly!
Extra:
Working with Windows 10 v22H2
DBeaver Version
Community edition 23.0.0
Operating System
Windows 10 v22H2
Database and driver
Postgres 14
Steps to reproduce
1.- Update from 22.3.1 to 23.0.0
2.- Open ERD diagram builded with 22.3.1
3.- Tables are gone
Additional context
Working with postgres local database
No proxy

Sorry, here´s the lines i've changed in the erd file with notepad
'  (from v22.3.1) '
'  ' (from v23.0.0) '

data-source id="postgres-jdbc-18587b23bf3-1fd32db3bf3e1b4c" (from 22.3.1)
changed with
data-source id="postgres-jdbc-186bc7c206a-426452715cf88d0" (from 23.0.0)

i am having problems same, the options from dump and restore db are not display. for you same? @mchable  ty

Hi tridoxx
I did not check that options, sorry i was in hurry trying to display my
diagrams but you can add the comment in the github board
El mié, 8 mar 2023 a las 18:03, tridoxx @.***>)
escribió:

i am having problems same, the options from dump and restore db are not
display. for you same? @mchable https://github.com/mchable ty
—
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/19302#issuecomment-1461006678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOO4U4O7GAKAMMZYSC7AOPLW3EF4ZANCNFSM6AAAAAAVUF64YA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi tridoxx I did not check that options, sorry i was in hurry trying to display my diagrams but you can add the comment in the github board El mié, 8 mar 2023 a las 18:03, tridoxx @.>) escribió:
…
i am having problems same, the options from dump and restore db are not display. for you same? @mchable https://github.com/mchable ty — Reply to this email directly, view it on GitHub <#19302 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOO4U4O7GAKAMMZYSC7AOPLW3EF4ZANCNFSM6AAAAAAVUF64YA . You are receiving this because you were mentioned.Message ID: @.>

hi ty, sorry i forget close and start the app, after update it was the error, so it was i deleted the comment. but ty

Hi. Thanks for answer.
I did not change my connections, but i will have that in mind in future updates.

Morning.
I did not at least in this update.
I'll check this appointment in the next update.

I'm sure it has something to do with connection duplication, etc. Maybe you just forgot about it.
We want to improve current behavior and warn users that the connection has associated resources (such as scripts, ERDs, and bookmarks) upon deletion.

Hi ShadelessFox.
Maybe if you attach a short message about a recommendation to backup the
connections file before update so they can restore it just in case. That's
what i decided to do for avoiding to lose my ERD's.
Thanks for your attention. Have a nice day.
El jue, 23 mar 2023 a las 8:34, ShadelessFox @.***>)
escribió:

I'm sure it has something to do with connection duplication, etc. Maybe
you just forgot about it.
We want to improve current behavior and warn users that the connection has
associated resources (such as scripts, ERDs, and bookmarks) upon deletion.
—
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/19302#issuecomment-1481205201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOO4U4OY35BKBHGP5XKFXJTW5RGN5ANCNFSM6AAAAAAVUF64YA
.
You are receiving this because you were mentioned.Message ID:
@.***>

It shouldn't be necessary. Data source IDs don't change on their own. You may have copied your data sources around and deleted the old ones that have had associated ERDs with them. Who knows.


marvin: add missing default test data

This add missing test data for one of the keys for a recently added
migration test.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

test_migration.py passes with this, on local environment. Tested this on a monkeybox env.
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2456

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3206)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26061 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3055-t3206-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 68 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

Ping @rafaelweingartner @wido @GabrielBrascher anyone?


⚠ Bofrost.nl has degraded performance

In df136eb, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1418 ms

Resolved: Bofrost.nl performance has improved in c839b06 after 6 minutes.


Remove the extension traits for BodyReader

Removes the unnecessary extension traits that provided extra parsing methods for BodyReader. These has instead been implemented directly.
This will break any project that uses these extension traits.
Closes #76

@homu r+

:pushpin: Commit c8da029 has been approved by Ogeon

:zap: Test exempted - status

Hmm, this broke my code! Does simply removing use rustful::context::body::ExtQueryBody; suffice?

Yes, the methods have moved to BodyReader, itself.

Ah I see, thanks!
By the way, it's been a great pleasure to use Rustful. Thanks for the great support. :smile:

Thank you! That's really nice to hear :smile:


Roslyn analyzers + code fixes don't work properly in VS when installed into .NET Core project via a nuget package

Originally posted at: https://github.com/dotnet/core/issues/91 by @HellBrick
(I can't seem to find a repo for raising VS .NET Core tooling issues, so if there's a more appropriate place for this, please tell and I'll be glad to repost it there.)
Here's what happens when a nuget package that contains Roslyn analyzers + code fixes is installed into a .NET Core project via install-package $(AnalyzerPackage):

Analyzer package is added to References > .NETStandard node of Solution Explorer instead of References > Analyzers.
Which also means there's no UI for changing rule severity.
Analyzers are not invoked when the code is being edited. The warnings are reported only when the project is built.
Even then no squiggles are displayed at the warning location, the warnings are only surfaced in the error list.
And probably the most tragic part: the code fixes attached to the analyzer warnings simply can't be triggered.

1 and 2 should get fixed once we implement #88 and 3,4,5 should get fixed once we implement #131

This is now fixed.


Modifications to master.cf

I haven't been able to find any build-in way to edit the master.cf in this image?
I'm in need of modifying it to allow submission of email on port 587 (Adding permit_mynetworks to client and recipient restrictions) and doing so manually is error-prone.
Does anyone have a way of doing this with this image?
If not, I'm preparing a PR allowing one to add a postfix-master.cf containing the config, in postconf -P format, just like adding the main.cf.

I haven't seen any way to accomplish this yet

Referencing your PR https://github.com/tomav/docker-mailserver/pull/595

I'll ret to finish the integration test in the coming days - Time is in short supply at the moment though.

On 16 May 2017, at 12.33, Thomas VIAL<EMAIL_ADDRESS>wrote:
Referencing your PR #595
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks, no problem!

@MadsRC this issue can be closed?

Should be resolved by https://github.com/tomav/docker-mailserver/pull/595


Warning: Failed loading manifest for /home/werner/.luarocks/lib/luarocks/rocks: /home/werner/.luarocks/lib/luarocks/rocks/manifest: No such file or directory

Hi all,
When I run the ``luarocks list'', I meet the following warning:
werner@debian-01:~$ luarocks list
Warning: Failed loading manifest for /home/werner/.luarocks/lib/luarocks/rocks: /home/werner/.luarocks/lib/luarocks/rocks/manifest: No such file or directory
Installed rocks:
luafilesystem
1.6.3-2 (installed) - /usr/local/lib/luarocks/rocks
luaposix
33.4.0-1 (installed) - /usr/local/lib/luarocks/rocks
luarocks
2.3.0-1 (installed) - /usr/local/lib/luarocks/rocks
luasocket
3.0rc1-2 (installed) - /usr/local/lib/luarocks/rocks

I think it's OK because after every manifest search it checks for errors and print warning message
      if not ok then
         util.warning(err)
      end

so first find local or remote manifest, then installed rocks, but I could be wrong

[1]  How to suppress this waring or make the manifest?
[2]  Even this warning exists for my case, it doesn't effect the installation of any modules, so it's harmless, am I right?
Regards

@robooo That's correct. Looking back, I think it would be nicer if we supressed the error in the "file not found" case (because that always happens when the user first installs LuaRocks and ~/.luarocks doesn't exist yet), but it would be good to report on any other errors (such as a corrupted manifest). We'd probably need a third return result (nil, err, errcode) so we could test something like if errcode == "missing" and not depend on the string of the message.
@hongyi-zhao
[1]:
mkdir -p ~/.luarocks/lib/luarocks/rocks
bin/luarocks-admin make-manifest --local-tree --tree=$HOME/.luarocks

[2] Yes, it's hamless!


Revoke kubeconfig file and re-create the new one

Hi team,
Currently, i want to revoke a kubeconfig file for some reason in a k8s installed by rke2. I did follow the doc to rotate the CA certificate but the old kubeconfig file still works.
Please support me

Hi @hdong69 , i don't think this github project is a right place for your question. Try it here https://github.com/rancher/rke2/discussions


Node providers that have multiple credentials exposes the full API Key when viewing them in Cloud Credentials

Setup

Rancher version: v2.6-head
Browser type & version: Chrome

Describe the bug

When looking into Cloud Credentials in Cluster Management, I noted that the API Key isn't hidden. For example, if you have Amazon credentials saved into Rancher from a previous cluster, it will list the entire access key, which isn't ideal.
However, when you look at a node driver such as Linode that only has one credential parameter, it will partially list the API Key and then finish off with ... to indicate that there is more. For cohesiveness and security purposes, we should apply that same approach Linode does to all of the cloud credentials.
In other words, we list the first 8 characters and afterwards have .... This is beneficial in the following ways: it hides the entire API Key from being visible, allows users to still manage their API Keys by having the first 8 characters as reference, gives a cohesive feel for all of the providers.
This is more of a "nice-to-have" feature as is by no means disrupting any workflows. But something that we should still be aware of.
To Reproduce

Setup Rancher and navigate to the UI in a browser.
Navigate to Cluster Management > Cloud Credentials.
Create an Amazon cloud credential and note the full API Key is shown.
If possible, create a Linode cloud credential. Note that only partial API Key is shown. All providers should match this.

Result
The full API Key is exposed when using providers such as Amazon that use multiple credentials to authenticate the user.
Expected Result

Only 8 characters should be exposed when providers such as Amazon use multiple means to authenticate the user. Currently, only providers that provide a single credential give this functionality.

While this issue is awaiting triage, here is the QA test plan:
TEST PLAN

Setup Rancher and navigate to the UI.
Create a standard user and login as that user.
Navigate to Cluster Management > Cloud Credentials.
Create an Amazon cloud credential; validate if the full API key is shown.
Repeat step 4 for providers Linode, Azure and Google; validate if the full API key is shown.

Per triage discussion, Linode only has a single key whilst Amazon does have two: the access key which tends to be more broadly visible and then the secret key which should always be hidden. We could potentially hide both and it would be trivial.


Use 'ping database' as readiness and liveness probe for each database

In this way, pod will be running only after the readiness probe is successsful (ie, DB is listening). So, setting DB.Status.Phase: Running will be more accurate and meaningful.
TODO:

[ ] Elasticsearch
[ ] Postgres
[ ] Mysql
[ ] Redis
[ ] Memcached

Example: https://github.com/kubedb/mongodb/blob/master/pkg/controller/statefulset.go#L205

@the-redback I think it is better to make the liveness /readiness probe configurable.

By confogurable, do you mean user should be able to enable or disable it? or, user can provide the full probe?

user can provide the full probe which can override the default probe . what do you think?

I think the way to implement this will be to use the new PodReady++ feature.

https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-readiness-gate
https://github.com/kubernetes/enhancements/blob/master/keps/sig-network/0007-pod-ready%2B%2B.md

From @stormmore
https://kubernetes.slack.com/archives/C8149MREV/p1537037429000100

most of  the time I am really only concerned with remote :slightly_smiling_face: I trust KubeDB enough to make sure it is up and ready on it's side the remote execution is so the other service will wait for KubeDB to say it's ready before starting
But for the record, as a general rule I don't believe a DB is ready until after any initialization scripts are completely done and hope that KubeDB backed DBs wouldn't necessarily response until then

Advanced Health Check Patterns in Kubernetes


Failed to inject player on version 1.18

Hi,
Under 1.18 server the plugin constantly comes back with "Failed to inject player [playername] please contact..." message. Easy to recreate the problem, just launch an 1.18 paper server and the plugin will fail. Will it still work though or if the injection is unsuccessful it will not work at all?

HamsterAPI is not yet compatible with 1.18.
It will probably be implemented in the future, but not at the moment as i don't have too much time available because of work.


Use of localStorage crashes Chrome when cookies aren't whitelisted

Describe the bug
When I open https://opencollective.com/eslint, I very briefly see the page. It then flashes to white with the browser tab spinning forever. If I leave it like this for too long, I have to force quite the browser tab via Chrome's "Task Manager".
I suspect this is due to an infinite loop where part of the code handling the problem itself tries to use localStorage. I see in the bottom-left corner that it tries it is accessing sentry.io many times in row.
To Reproduce
Steps to reproduce the behavior:

Use the current stable version of Chrome .
Open Settings > Advanced > Site settings > Untick "Allow sites to save and read cookie data".
Open https://opencollective.com/eslint
Observe the above problem.

Expected behavior
For users to be able to at least browse pages on https://opencollective.com without demanding that information about them or from your website is permanently stored on their computer. I'm well aware that many features may require cookies, that's no problem. I have a separate browser profile with limited expiry and no such restriction in which I browse such websites. I would like however to be able to at least navigate and peruse websites like https://opencollective.com in my main browser. If I have to whitelist this website in order to sign in or may payments, or do so through my other browser profile, that would be totally fine.
Screenshots

Desktop:

OS: macOS
Browser: Chrome
Version: 77

Additional context
I believe this is related to https://github.com/opencollective/opencollective-frontend/pull/2642 which attempted to fix a similar issue, but didn't quite get it. It's pretty rare in modern browsers for localStorage to be null (even in search engines, cookie-restriction modes, or private browsing modes). Rather, the object tends to exist but throw on access.

you need to give chrome permission to work ,or your running It while in
roaming..
also u could clear ur storage..try the chromium projects..it's alot bettrr
On Sat, Oct 19, 2019, 11:40 AM Timo Tijhof<EMAIL_ADDRESS>wrote:

Describe the bug
When I open https://opencollective.com/eslint, I very briefly see the
page. It then flashes to white with the browser tab spinning forever. If I
leave it like this for too long, I have to force quite the browser tab via
Chrome's "Task Manager".
I suspect this is due to an infinite loop where part of the code
handling the problem itself tries to use localStorage. I see in the
bottom-left corner that it tries it is accessing sentry.io many times in
row.
To Reproduce
Steps to reproduce the behavior:

Use the current stable version of Chrome .
Open Settings > Advanced > Site settings > Untick "Allow sites to
save and read cookie data".
Open https://opencollective.com/eslint
Observe the above problem.

Expected behavior
For users to be able to at least browse pages on
https://opencollective.com without demanding that information about them
or from your website is permanently stored on their computer. I'm well
aware that many features may require cookies, that's no problem. I have a
separate browser profile with limited expiry and no such restriction in
which I browse such websites. I would like however to be able to at least
navigate and peruse websites like https://opencollective.com in my main
browser. If I have to whitelist this website in order to sign in or may
payments, or do so through my other browser profile, that would be totally
fine.
Screenshots
[image: 1]
https://user-images.githubusercontent.com/156867/67149818-c7611780-f2a7-11e9-9337-a7ba545702ac.png [image:
2]
https://user-images.githubusercontent.com/156867/67149819-c7611780-f2a7-11e9-816a-006481cd7699.png
[image: 3]
https://user-images.githubusercontent.com/156867/67149820-c7f9ae00-f2a7-11e9-9781-de66379c22f8.png [image:
4]
https://user-images.githubusercontent.com/156867/67149821-c7f9ae00-f2a7-11e9-87b6-4082c446cca7.png
Desktop:

OS: macOS
Browser: Chrome
Version: 77

Additional context
I believe this is related to opencollective/opencollective-frontend#2642
https://github.com/opencollective/opencollective-frontend/pull/2642
which attempted to fix a similar issue, but didn't quite get it. It's
pretty rare in modern browsers for localStorage to be null (even in
search engines, cookie-restriction modes, or private browsing modes).
Rather, the object tends to exist but throw on access.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opencollective/opencollective/issues/2542?email_source=notifications&email_token=AM66E63C4NISDDKAF3OW2UTQPNIBDA5CNFSM4JCQ6Q3KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HS6H53A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM66E6Y5NEEOEL6HRRBUJMLQPNIBDANCNFSM4JCQ6Q3A
.


Debug output timestamps are in UTC

Can you please either change it to local time, or add an option?
I'm using VS2019 latest (16.4.3)

something something brain-fart


Communicate to R&D that bpm necessitates blackbox

Per CloudOps PWS learnings, communicate that BPM requires blackbox to continue forwarding logs to syslog. Additionally, if BPM-managed jobs are co-located with jobs that use traditional ctl scripts, we need a custom rule to filter avoid double-logging from ctl-script jobs.
https://www.pivotaltracker.com/story/show/157578210

Looks good! I left a few small comments

updated this PR according to the comments above.

lgtm


Added ability to work with custom certs

Description
Some organisations uses their own cacert in their network so I've added ability to either introduce their cacert to service or ignore it totaly.
Type of change

[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

How Has This Been Tested?
Test was done with passing different type of configs to the code and do api calls.
Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[X] Do all new files have appropriate license header?

@etrexel had a suggestion to mount the trusted cert instead of adding logic in the application code. @etrexel could you elaborate on mounting the cert to achieve the same result?

In the past when I have worked with custom CAs in Kubernetes I have followed a pattern of including the custom CA in the host bundle and hostPath mounting /etc/ssl/certs as readonly over the container's /etc/ssl/certs so that applications don't have to be custom CA aware. I think it would be reasonable to suggest that approach in this case as the host would need the custom CA anyways to be able to pull the container.
Instead of updating the application to handle custom CAs we can just update the chart to conditionally mount the host cert bundle into the container. We will have to validate the correct path for certs in distroless.
I don't know how this would impact Windows clusters since Windows handles certificates differently, but I don't think we have near-term plans to publish Windows container images of Ratify.
Do you have any concerns with this approach?

Hi everyone, Thanks for explanation. I'll give it try and reach back to you.

Closing for now as PR is stale. Feel free to reopen if further concerns.


Get the number of chapters in a book

Add the ability to get the number of chapters in a bible book. This can help with scraping

Thx Jude & Jon


guidelines: move CMN examples to Verovio

This PR brings most of the encodings back that were originally used to render the examples with MEILER, now to be rendered live with Verovio.
Unfortunately Verovio doesn't show all encodings as expected yet.
closes #1160

Still needs work. Moving to 6


fix: TrimEditor error: 'bytes != null': is not true #95

add thumbnail cache when get null use cache

This fixes #95 and #94 and related. The issue still exists so this commit should be merged. However, make sure apply the suggested change.

Closing this. I have merged the null safety version of this.


feat: add ctor error frame to stack trace

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.45%. Comparing base (0c3eb72) to head (f27a190).

Additional details and impacted files
@@                        Coverage Diff                         @@
##           dori/change-ctor-error-variant    #1845      +/-   ##
==================================================================
- Coverage                           77.45%   77.45%   -0.01%
==================================================================
  Files                                  60       60
  Lines                                8579     8577       -2
  Branches                             8579     8577       -2
==================================================================
- Hits                                 6645     6643       -2
  Misses                               1494     1494
  Partials                              440      440

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


.JMAlert(showModal: x, for: [y], restrictDismissal: z) not an option

Overview
There is no JMAlert with option to restrictDismissal. There IS one for JMModal, but I want to use JMAlert.
Action items
Option for .JMAlert(showModal: self.$showModal, for: [.notification, .tracking], restrictDismissal: false)

@smaldd14 Really??! This is interesting, it should be exactly same for both, I will get this fixed.

Also noticed this after switching from modal to alert.

@xoniq @smaldd14 Please try v1.5.4


Store is not initialized until state observable is subscribed

If I don't subscribe the state observable (or don't call it through  an async pipe), the state store isn't initilized.
Steps to reproduce:

Comment line 2 on app.component.html from ng-sa-store-services example (pull request sent) and observe Redux devtools to inspect state change when the app load and after, when the toggle button is pressed.

Workaround: Always subscribe the state oberver to initialize the store.

This is intentional, but not 100% settled. Originally state would initialize as soon as adapt.init was called. But the issue with that is I would need to provide another method for cleaning up dynamically created states to avoid memory leaks, so I decided to just hook everything together in a single observable chain and let finalize() take care of cleaning up the dynamically created state.
I'm not 100% settled on this. While developing, it is easy to just throw a .subscribe() into the constructor. After that, my theory was that the state only matters if something in the UI is going to show something from it, so there would always be some async pipe to hold the state there. I am looking for counter-examples of this, but each time I think of one it makes way more sense to show something from the state in an async pipe.
In the shopping cart example, you might add something to the cart and if the cart is not open, then that's a problem, right? Yet, the button to Add to Cart is interested in what is in the cart already, isn't it (so it can show if the item is already in the cart)? And even if it isn't, it seems like every website with a shopping cart shows the total number of items in a badge on the cart icon.
I don't know every possible app that will be created, so I could imagine a scenario where somebody needs to perform repeated actions that affect some completely hidden app state. I do not love the idea of making people subscribe() just to hold the state in place. I'm even more worried about the frustration when people expect state to update but nothing happens. I can try to tell people to think of stores as scan() operators and NOT like BehaviorSubjects, but it seems like every framework has its landmine for beginners, and this would be it for StateAdapt.
If I emphasize that init is like scan every time I talk about it, maybe that will be enough. I did mention in my introduction article that nothing happens until you subscribe: StateAdapt is reactive, so everything is lazy, like RxJS. So far nothing will actually happen with any state, even if events are pushed into our sources, because nothing is actually subscribing to anything yet.
Did you read that article? If you did, then I expect a lot of people to forget this part and maybe I should think seriously about changing the default behavior.

@mfp22 Ok, my mistake. I've read the article but I haven't paid enought attention to the details. Said that, I just finished reading again and the part of "everythin is lazy" is very subtle for me, due to the facts I had my mind shapped by NgRX behavior.
I have expected whenever I send events, the state store be initialized, but now I understand the state adapter way.
In my particular case, I have events on app.component to deal authentication and guards and other modules or components observe these state changes. So the change events isn't near to observers and the state store is used as a common bus among components. This is one scenario where the actions dispatch could initialize the store, but call the subscribe method is not terrible.
Another parts I wasn't sure if you were talking about internals details or way of use, so I think a demo covering all aspects aborded wil be great, mainly the use of asyncAdapter and custom selectors.


[firebase_auth plugin] README.md Updated

Description
I had just updated the README.md file. There isn't any deletion of any pre-existed info in README.md
Related Issues
There isn't any issue created for this, but I thought it would be very helpful to add some examples of doing Firebase Facebook and Firebase Twitter authentication.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@chinkysight Thanks for the PR, would you mind rebasing this so that the tests can pass and I'll be happy to review.


Network window extends when networks are loaded

As seen in pictures the network tab window extends a lot when networks are loaded. I think it should probably be changed to not do that. I think it would be better to not contain so much information in navigational elements. I think we should move this to a separate page and the navigation element can only hold summary, so if all things are green it will contain green icon saying everything is fine, if one is red then it contain the one that's not ok and an orange icon and see more for details button that takes you to the mentioned page.
How it currently expands

static builds https://github.com/onflow/docs with a link to network status